<joseph_>
kintel: Right, so I've been thinking that one of the earliest targets of refactoring is unifying the rendering logic between OpenCSGRenderer and CGALRenderer. Helpfully, many of my modifications to CGALRenderer last summer are mostly copied from what was already in OpenCSGRenderer. But it seems to me that I haven't yet touched the code that is actually causing the artifacts: the fixed-function drawing of OpenCSG primitives. I can
<joseph_>
start searching for where that happens, unless you know off the top of your head where it's located
J23k4 has joined #openscad
J23k59 has quit [Ping timeout: 245 seconds]
<peeps[zen]>
guso78: that image is manifold as long as the front face has 5 sides (2 colinear at the top edge). every edge consists of two half-edges, each half-edge belongs to exactly one polygonal face
KimK has joined #openscad
fling has quit [Ping timeout: 240 seconds]
<peeps[zen]>
that's valid AFAIK in CGAL, assuming all the 5 points on the front face are coplanar. i think Manifold library requires triangulation, which would require J23k4's edited version of the image.
<peeps[zen]>
but the necessary triangulation should happen behind the scenes as long as it was valid as previously described
<peeps[zen]>
BTW, J23k4 all your links add an underscore that breaks things and requires manually editing url for me
<kintel>
My code only touches the SCS algorithm; I didn't yet look into Goldfeather
<kintel>
Btw. in terms of OpenGL modernization, one approach to tease out all the call sites could be to start with https://github.com/openscad/openscad/pull/4582 and request an OpenGL 3 context without compatibility profile. The EGL code isn't finished, but it should work well on macOS
foul_owl has joined #openscad
ali1234 has joined #openscad
ali12341 has joined #openscad
ali12341 has quit [Client Quit]
<joseph_>
Thanks for the advice. I'll investigate
use-value has joined #openscad
kintel has quit [Quit: My MacBook has gone to sleep. ZZZzzz…]
<peeps[zen]>
that one is fine. idk, your imgpile link from earlier was messed up though, and i noticed it a couple other times hadn't bothered mentioning it until now.