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

Faulty Simplify Quadric Decimation since 0.18.0. 0.17.0 seems to work more reliable. #6673

Closed
3 tasks done
limpbot opened this issue Mar 1, 2024 · 6 comments
Closed
3 tasks done
Labels
bug Not a build issue, this is likely a bug.

Comments

@limpbot
Copy link

limpbot commented Mar 1, 2024

Checklist

Describe the issue

Using 0.18.0 simplify_quadric_decimation does not reach the target_number_of_triangles reliable for the given not watertight mesh. In 0.17.0 this works reliable.

Probably related to #6163 .

Steps to reproduce the bug

import open3d

mesh = open3d.io.read_triangle_mesh('not_watertight_mesh.ply')
print(mesh)
mesh_decimated = mesh.simplify_quadric_decimation(target_number_of_triangles=mesh_vertices_count)
print(mesh_decimated)

Error message

faulty_simplify_quadratic_decimation

Expected behavior

No response

Open3D, Python and System information

- Operating system: Ubuntu 22.04
- Python version: Python 3.10
- Open3D version: output from python: 0.18.0
- System architecture: x86 
- Is this a remote workstation?: no
- How did you install Open3D?: pip

Additional information

No response

@limpbot limpbot added the bug Not a build issue, this is likely a bug. label Mar 1, 2024
@limpbot
Copy link
Author

limpbot commented Mar 1, 2024

@ssheorey
Copy link
Member

ssheorey commented Mar 7, 2024

@bjudeworley can you take a look at this? Thanks!

@bjude
Copy link

bjude commented Mar 12, 2024

I'll try and take a look but I now have a 3 month old so my free time is basically nil so it might be a while.

If I had to guess, the 0.17.0 output probably has one of the bad element types my PR aimed to fix, or requires generating them at some intermediate step in the decimation process. Maybe take a look at the output mesh in something like Meshmixer and see if that's the case.

@limpbot, your example doesnt have enough info, what is the value of mesh_vertices_count?

@limpbot
Copy link
Author

limpbot commented Mar 12, 2024

hi @bjude, thx for taking a look. The vertices count should be 7055, and the faces count should be 15670.

@bjude
Copy link

bjude commented Mar 13, 2024

@limpbot can you re-upload the mesh, its 404'ing when i try and download it

@limpbot
Copy link
Author

limpbot commented Mar 21, 2024

Sorry, I have also problems loading it currently. Will reopen if I find another example

@limpbot limpbot closed this as completed Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Not a build issue, this is likely a bug.
Projects
None yet
Development

No branches or pull requests

3 participants