Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

export with compress on cause uv error #2279

Open
zhou1993yu opened this issue Jul 2, 2024 · 3 comments
Open

export with compress on cause uv error #2279

zhou1993yu opened this issue Jul 2, 2024 · 3 comments
Labels
bug Something isn't working draco

Comments

@zhou1993yu
Copy link

Describe the bug
export with compress on cause uv error.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'file->export->gltf 2.0'
  2. with compress on and press export
  3. reimport the exported model
  4. See error below

Expected behavior

Screenshots
20240702104214
20240702104352

.blend file/ .gltf (mandatory)
untitled.zip

Version

  • OS: windows
  • Blender Version 4.0 ~4.2

Additional context
Add any other context about the problem here.

@julienduroure
Copy link
Collaborator

I can confirm.
Bug is visible in online viewer like https://gltf-viewer.donmccurdy.com/, so problem comes from exporter

@julienduroure julienduroure added bug Something isn't working draco labels Jul 3, 2024
@donmccurdy
Copy link
Contributor

donmccurdy commented Jul 27, 2024

If you zoom out considerably in the UV Editor view, on the uncompressed mesh, you'll see a small outlying UV island —

CleanShot 2024-07-26 at 21 51 15@2x

This will break quantization, which both Draco and Meshopt compression rely on — most of the available precision in the quantization grid is spent on empty space. I'm not sure how UVs end up there to begin with, but it's been a recurring issue in glTF Transform's compression as well:

I've been considering adding a step to compute a histogram for the UVs, and to show a warning if there are major outliers, or perhaps repacking the UVs, but these are potentially expensive operations for an exporter to spend time on...

@julienduroure
Copy link
Collaborator

Thanks @donmccurdy for the investigation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working draco
Projects
None yet
Development

No branches or pull requests

3 participants