Summary: | KAlgebra 3D graphs are not plotted on the PinePhone due to shader error | ||
---|---|---|---|
Product: | [Frameworks and Libraries] analitza | Reporter: | Kevin Kofler <kevin.kofler> |
Component: | analitzaplot | Assignee: | Aleix Pol <aleixpol> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | percy.camilo.ta |
Priority: | NOR | ||
Version First Reported In: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Manjaro | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/education/analitza/commit/2af6c7ddbd5d961dc0aefda0e08367fb2cf1d858 | Version Fixed In: | 22.12 |
Sentry Crash Report: |
Description
Kevin Kofler
2022-10-11 13:40:18 UTC
And in case it matters: Kernel version: 5.19.11-1-MANJARO-ARM And I am using the qt5-es2-{base,declarative,multimedia} (OpenGL ES) builds. Looks like the offending code is actually part of analitza(plot). I am using: analitza 22.08.1-1 Apparently a regression from this 2015 commit: https://invent.kde.org/education/analitza/-/commit/80a3833ee306ee62ef61126d31fd7649bc7394af I guess it should say "highp float w = …" instead of "float w = …". Ping? Aleix, can you please look into this? Or anybody else? It looks to me like this should be a one-word fix (see my comments above), though I am not by any means an expert for OpenGL shaders. Git commit 2af6c7ddbd5d961dc0aefda0e08367fb2cf1d858 by Aleix Pol. Committed on 03/11/2022 at 01:25. Pushed by apol into branch 'master'. analitzaplot: Fix 3D plots on the pinephone It complains about the lack of precision on the float there whereas elsewhere it doesn't. Adding the highp still works on other devices so we have all bases covered. M +1 -1 analitzaplot/plotter3d_es.cpp https://invent.kde.org/education/analitza/commit/2af6c7ddbd5d961dc0aefda0e08367fb2cf1d858 There you go, it was exactly what you said. Git commit 9f32552760eaf462a79356011ed117a4ce03a62a by Aleix Pol Gonzalez, on behalf of Aleix Pol. Committed on 03/11/2022 at 01:28. Pushed by apol into branch 'release/22.08'. analitzaplot: Fix 3D plots on the pinephone It complains about the lack of precision on the float there whereas elsewhere it doesn't. Adding the highp still works on other devices so we have all bases covered. (cherry picked from commit 2af6c7ddbd5d961dc0aefda0e08367fb2cf1d858) M +1 -1 analitzaplot/plotter3d_es.cpp https://invent.kde.org/education/analitza/commit/9f32552760eaf462a79356011ed117a4ce03a62a Thanks a lot! Unfortunately, this missed 22.08.3 by 2 days, and there is no 22.08.4 planned, so 22.12(.0) will likely be the first release to ship the fix. |