bugGNU Octave - Bugs: bug #46997, surface with "FaceColor"...

 
 

bug #46997: surface with "FaceColor" set to "texturemap" display incorrectly and issue an OpenGL warning

Submitted by:  Markus Mützel <mmuetzel>
Submitted on:  Thu 28 Jan 2016 12:36:08 PM UTC  
 
Category: Plotting with OpenGLSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: 4.0.0Operating System: Microsoft Windows

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 04 Oct 2016 08:07:59 AM UTC, comment #5:

The underlying issue was fixed in bug #45542 here:
http://hg.savannah.gnu.org/hgweb/octave/rev/3564b6b6b8d1

This report can now be closed as fixed.

Markus Mützel <mmuetzel>
Project Member
Fri 02 Sep 2016 07:32:21 PM UTC, comment #4:

This bug is no longer present for me with a current dev on Windows (hg id: 7e152b7c2a2c).

The report can be closed.

Markus Mützel <mmuetzel>
Project Member
Fri 22 Jul 2016 01:49:16 PM UTC, comment #3:

The OpenGL error in comment #0 is not actually caused while painting the texture. In fact, it looks like the error flag is already set before the texture is created.

The attached patch fixes the original issue for me.

However, this does not actually fix the issue which initially caused the OpenGL error flag to be set. Maybe we should check more often whether the error flag was set? This should probably be done at least at the end of every "opengl_renderer::draw_..." function or at the end of "opengl_renderer::draw". But that is probably better tracked in another bug.

(file #37988)

Markus Mützel <mmuetzel>
Project Member
Mon 08 Feb 2016 09:17:33 AM UTC, comment #2:

returns "qt".

The behavior is the same when I change the graphics toolkit to "fltk": The "oldest" surface with "FaceColor" set to "texturemap" is displayed with only one color and the same warning is displayed.

When I set the graphics toolkit to "gnuplot", the "oldest" surface set to "texturemap" is drawn with differently colored rectangles (as opposed to only one color when it fails) and no warning message is displayed. However, it looks like "texturemap" is not implemented at all for "gnuplot". In the example in 46997#comment0, I see surfaces with 9x9 colored rectangles - as would be correct for "FaceColor" set to "flat". With "FaceColor" set to "texturemap", it would be correct to see surfaces with 10x10 colored rectangles.
But I guess that is a different bug. (Maybe a warning should be issued in this case?)

I tried the same on a VM with Windows 10 with different results (but also failing). But as there is a warning during installation that Octave might probably fail on Windows 8 (and later?), I guess it makes no sense to report those, does it?

Please, let me know if I can help with any more infos or tests.

Markus Mützel <mmuetzel>
Project Member
Mon 08 Feb 2016 07:11:59 AM UTC, comment #1:

I can't reproduce this on Linux, either with fltk or qt.

What graphics toolkit are you using? Type

Use available_graphics_toolkits to see what other toolkits are supported in your system, and try one of those. Does it still fail?

Lachlan Andrew <lachlan>
Project Member
Thu 28 Jan 2016 12:36:08 PM UTC, original submission:

When I use surface to draw sections of 3d data with "FaceColor" set to "texturemap", the "oldest" surface with this option fails to draw correctly (only one color). The following warning is displayed in the command window:

Subsequent surfaces with this option set, draw correctly.

The warning message repeats everytime, I touch the figure (change view, add surface, zoom, ...).

Please, see the attached example. Starting line 21 is where it gets strange: The drawing error moves over to the oldest surface with "FaceColor" set to "texturemap"...

Markus Mützel <mmuetzel>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #37988:  texturemap.patch added by mmuetzel (819B - text/x-diff)
file #36191:  texturemap_test.m added by mmuetzel (1KiB - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by pantxo (Updated the item)
  • -unavailable- added by mmuetzel
  • -unavailable- added by lachlan (Posted a comment)
  • -unavailable- added by mmuetzel (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 04 Oct 2016 11:00:35 AM UTCpantxoStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Fri 22 Jul 2016 01:49:16 PM UTCmmuetzelAttached File-=>Added texturemap.patch, #37988
      Carbon-Copy-=>Added pantxo
    Thu 28 Jan 2016 12:36:08 PM UTCmmuetzelAttached File-=>Added texturemap_test.m, #36191

    Back to the top


    Powered by Savane 3.1-cleanup1