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

 
 

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

Submitter:  Markus Mützel <mmuetzel>
Submitted:  Thu 28 Jan 2016 12:36:08 PM UTC
   
 
Category:  Plotting with OpenGL Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 4.0.0 Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group administrator
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>
Group administrator
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>
Group administrator
Mon 08 Feb 2016 09:17:33 AM UTC, comment #2: 


graphics_toolkit

 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>
Group administrator
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


graphics_toolkit


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>
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:

warning: opengl_texture::create: OpenGL error while generating texture data


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>
Group administrator

 

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

Attach Files:
   
   
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
  • -email is unavailable- added by pantxo (Updated the item)
  • -email is unavailable- added by mmuetzel
  • -email is unavailable- added by lachlan (Posted a comment)
  • -email is unavailable- added by mmuetzel (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-10-04 pantxo StatusNone Fixed
        Open/ClosedOpen Closed
    2016-07-22 mmuetzel Attached File- Added texturemap.patch, #37988
        Carbon-Copy- Added pantxo
    2016-01-28 mmuetzel Attached File- Added texturemap_test.m, #36191

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code