guillaumechereau / goxel

Goxel: Free and Open Source 3D Voxel Editor
GNU General Public License v3.0
2.82k stars 225 forks source link

Bug: GLTF have erroneous vertices #336

Open jerobarraco opened 10 months ago

jerobarraco commented 10 months ago

i've been having a ton of issues with this since build #60 or so. once the colors got fixed.

this is for meshes that are NOT simplified. simplified meshes have far more errors and data loss, so no tweak in unreal engine can help

sometimes i can work around it, sometimes i cant. sometimes it affects the colors so the colors look bad (using vertex colors).

https://www.youtube.com/live/MPtNi802P-U?&t=869 https://www.youtube.com/live/MPtNi802P-U?&t=4134 https://www.youtube.com/live/MPtNi802P-U?&t=4468

image image

here are the files WordCube.zip

guillaumechereau commented 10 months ago

I'll have a look, but how do you make vertex color work in Unreal Engine? I could only make it work with textured color so far. By the way did you try the last nightly build, I added a fix for the textures in Unreal Engine.

jerobarraco commented 10 months ago

thanks, i havent tried it yet but ill try soon. i don't want to use textures. just use this node in a material image

previously i was doing a naive color correction but i dont seem to need it anymore. image

jerobarraco commented 10 months ago

i just tried build #165 nice new interface! i love you can now pin some tabs, that was driving me crazy. it looks a bit sharper, is that a new ui toolkit? also i liked the new cube at top/right.

also the slider on the simplify is amazing. though i'm still experiencing issues on the vertices being at odd places. places where there should be no vertex. it's an annoying bug. even at different compression levels.

in fact the issue exists at no simplification level at all (0). so i think it's a bug on the export not on the simplification.

jerobarraco commented 10 months ago

something interesting is that the error is deterministic (which is cool). i re-exported at different levels of compression, and when i re-exported back with no compression git detected it was the same file as the original.

jerobarraco commented 10 months ago

also i cant remember now, but the simplification does not affects the error. that is, even while changing the simplification the error in the output is exactly the same, which is nice, means that the simplification step is likely not the issue.

jerobarraco commented 10 months ago

also seems to happen on highly detailed objects. on walls and chairs it has been ok.

guillaumechereau commented 10 months ago

I need to check this into more detail. Don't hesitate to share models that have the biggest issue so that I can test them out.

guillaumechereau commented 10 months ago

For the new UI, I still use ImGui, but I customized it to make it look a bit better. Still a few issues there to fix out before the next release.

jerobarraco commented 5 days ago

still occurs but very sparingly. still happens on 1.51 and build 446

here's a case. image

Screenshot_20241107_200342

github does not allow me to attach a binary :shrug: otherwise i'll attach the model.