842 2013-09-25 00:53:09 +0200 GLJPanel's GLSL Image flip method requires glFrontFace to be GL_CCW 2014-01-25 00:39:07 +0100 1 1 3 JogAmp Jogl awt 2 All all RESOLVED WONTFIX --- minor --- 0 bkuker sgothel cmbruns gouessej --- ab860381009c0716a99d33f0ce99f44617edf9ce --- oldest_to_newest 2989 0 bkuker 2013-09-25 00:53:09 +0200 If you leave the front-face winding Clockwise (GL_CW) the glsl method to flip the image before reading it into the image raster fails to work. The screen is left full of garbage, or black depending the system. I assume the geometry that it draws the flipped image to ends up culled and nothing is drawn. I'd consider this minor because it is easy to work around, call gl.glFrontFace(GL.GL_CCW) at the end of display() and who changes the winding rule anyway! 2992 1 506 afterglow79 2013-09-26 21:31:58 +0200 Created attachment 506 Test case demonstrating this bug. GL_CW should be usable as is - without extra (undocumented) commands. This functionality is central to the application I am currently working on which renders semi-transparent objects, front and back. The solution of resetting glFrontFace to GL_CCW did work for me. 2999 2 sgothel 2013-09-29 21:15:27 +0200 (In reply to comment #1) > Created attachment 506 [details] > Test case demonstrating this bug. > > GL_CW should be usable as is - without extra (undocumented) commands. This > functionality is central to the application I am currently working on which > renders semi-transparent objects, front and back. The solution of resetting > glFrontFace to GL_CCW did work for me. For Bug 826, we started to preserve some states (viewport and texture) while utilizing the GLContext for an GLSL based vertical flip. The problem w/ the fixed function pipeline (ffp) is that there might be many states involved influencing the flipping. We could either disable it for ffp usage, or let it up to the user to take care of it, as Bill mentioned. I would vote for the latter and leave it as-is. But this is surely a matter of taste and if leaving as-is, we might add some special API doc warning the user upfront. Shall we discuss this issue with a broader audience ? Sure .. if somebody is willing to analyze the ffp states and is able to create such a helper class to preserve those states .. you are welcome. Until then .. I am undecided. 3004 3 bkuker 2013-09-29 22:06:44 +0200 I think the main trouble is surprise: In my case my program was totally failing when I switched from GLCanvas to GLJPanel and I ended up writing a test program from scratch to convince myself that GLJPanel actually worked at all, and then started commenting out half of my code in display() at a time until I narrowed it down to the right thing. This bug is very easy to work around once you know what you need to do, but it is very hard to know what you need to do. 3011 4 gouessej 2013-10-01 10:33:55 +0200 I agree with Mike. In the worst case, it should be documented. I would prefer all states to be preserved in the fixed function pipeline and we could refine this mechanism later if it is too slow. A developer switching from GLCanvas to GLJPanel should not waste tons of time in working around undocumented state changes even though JOGL is open source. The test case mentioned by Mike is quite typical in scientific visualization, I had to do something similar when displaying some geological layers. The current situation is not satisfying. Please Sven, drive those limitations transparent for the developers or document them. Of course I prefer the former solution. 3027 5 sgothel 2013-10-02 16:37:08 +0200 Added 'Warning' regarding GLSL vertical flipping - also added notice about how we preserve texture-unit states and the viewport. I am aware that probably nobody will read those details, but at least it is in the documentation now. I also realize that the mentioned state is a GL-core feature, so not restricted to the FFP. Alternatively we could use a shared GL context (performance loss while switching), or drop GLSL flip all-together (performance loss). I close this bug for now (WONTFIX) - if you like to re-open it to pursue one of the above alternatives or another (i.e. preserve all the states) .. please do so. I will merge your changes ofc. 3028 6 bkuker 2013-10-02 16:50:35 +0200 (In reply to comment #5) I think its fine to not fix it, the problem is confirmed understood and documented, and the workaround is easy. I think it does deserve a mention in the GLJPanel javadoc. To be honest I'll choose to be a little selfish and write a one line workaround rather than a patch to save / restore all states. 3645 7 sgothel 2014-01-25 00:38:07 +0100 Just for completeness: Best GLJPanel solution for performance and compatibility is GLJPanel's 'setSkipGLOrientationVerticalFlip(true)', see Bug 904, and referenced git-sha1 commits (incl. unit tests). 3647 8 sgothel 2014-01-25 00:39:07 +0100 *** Bug 874 has been marked as a duplicate of this bug. *** 506 2013-09-26 21:31:58 +0200 2013-09-26 21:31:58 +0200 Test case demonstrating this bug. OneTriangle.java application/octet-stream 1396 afterglow79 cGFja2FnZSBqb2dsLmJhc2UudGVzdDsNCg0KaW1wb3J0IGphdmF4Lm1lZGlhLm9wZW5nbC5HTDsN CmltcG9ydCBqYXZheC5tZWRpYS5vcGVuZ2wuR0wyOw0KaW1wb3J0IGphdmF4Lm1lZGlhLm9wZW5n bC5nbHUuR0xVOw0KDQpwdWJsaWMgY2xhc3MgT25lVHJpYW5nbGUgew0KICAgIHB1YmxpYyBzdGF0 aWMgdm9pZCBzZXR1cCggR0wyIGdsMiwgaW50IHdpZHRoLCBpbnQgaGVpZ2h0ICkgew0KICAgICAg ICBnbDIuZ2xNYXRyaXhNb2RlKCBHTDIuR0xfUFJPSkVDVElPTiApOw0KICAgICAgICBnbDIuZ2xM b2FkSWRlbnRpdHkoKTsNCg0KICAgICAgICAvLyBjb29yZGluYXRlIHN5c3RlbSBvcmlnaW4gYXQg bG93ZXIgbGVmdCB3aXRoIHdpZHRoIGFuZCBoZWlnaHQgc2FtZSBhcyB0aGUgd2luZG93DQogICAg ICAgIEdMVSBnbHUgPSBuZXcgR0xVKCk7DQogICAgICAgIGdsdS5nbHVPcnRobzJEKCAwLjBmLCB3 aWR0aCwgMC4wZiwgaGVpZ2h0ICk7DQoNCiAgICAgICAgZ2wyLmdsTWF0cml4TW9kZSggR0wyLkdM X01PREVMVklFVyApOw0KICAgICAgICBnbDIuZ2xMb2FkSWRlbnRpdHkoKTsNCg0KICAgICAgICBn bDIuZ2xWaWV3cG9ydCggMCwgMCwgd2lkdGgsIGhlaWdodCApOw0KICAgIH0NCg0KICAgIHB1Ymxp YyBzdGF0aWMgdm9pZCByZW5kZXIoIEdMMiBnbDIsIGludCB3aWR0aCwgaW50IGhlaWdodCApIHsN CiAgICAgICAgZ2wyLmdsQ2xlYXIoIEdMLkdMX0NPTE9SX0JVRkZFUl9CSVQgKTsNCg0KICAgICAg ICBnbDIuZ2xFbmFibGUoR0wyLkdMX0NVTExfRkFDRSk7DQogICAgICAgIGdsMi5nbEZyb250RmFj ZShHTDIuR0xfQ1cpOwkvLyB1c2luZyB0aGlzIGN1bGxpbmcgb3B0aW9uIHdpbGwgY2F1c2UgdGhl IHRyaWFuZ2xlIG5vdCB0byBiZSBkcmF3bg0KICAgICAgICANCiAgICAgICAgLy8gZHJhdyBhIHRy aWFuZ2xlIGZpbGxpbmcgdGhlIHdpbmRvdw0KICAgICAgICBnbDIuZ2xMb2FkSWRlbnRpdHkoKTsN CiAgICAgICAgZ2wyLmdsQmVnaW4oIEdMLkdMX1RSSUFOR0xFUyApOw0KICAgICAgICBnbDIuZ2xD b2xvcjNmKCAxLCAwLCAwICk7DQogICAgICAgIGdsMi5nbFZlcnRleDJmKCAwLCAwICk7DQogICAg ICAgIGdsMi5nbENvbG9yM2YoIDAsIDEsIDAgKTsNCiAgICAgICAgZ2wyLmdsVmVydGV4MmYoIHdp ZHRoLCAwICk7DQogICAgICAgIGdsMi5nbENvbG9yM2YoIDAsIDAsIDEgKTsNCiAgICAgICAgZ2wy LmdsVmVydGV4MmYoIHdpZHRoIC8gMiwgaGVpZ2h0ICk7DQogICAgICAgIGdsMi5nbEVuZCgpOw0K ICAgICAgICANCiAgICAgICAgLy9nbDIuZ2xGcm9udEZhY2UoR0wyLkdMX0NDVyk7CS8vIEhPV0VW RVIsIFRISVMgTVVTVCBCRSBVTkNPTU1FTlRFRCBGT1IgVEhFIFNDUkVFTiBOT1QgVE8gQkUgRklM TEVEIFdJVEggR0FSQkFHRQ0KICAgIH0NCn0NCg==