-
Notifications
You must be signed in to change notification settings - Fork 4
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
check_watertight needs default value of tolerance to check model for water tightness #5
Comments
Wouldn't we just use the faceting tolerance as taken from the .h5m file. Or perhaps an order larger? (10xfacet_tol?) |
Yes, or possibly down to dags facet intersection tolerance 1.e-6 |
I would be ok with that. I'd like it working as robustly as possible. |
Issue handled. There are two ways in which check_watertight works. One in which is uses the proximity facet vertices to ensure that they are within some tolerance of each other (the tolerance being specified by the user after the -f flag in the command), and the other is a topological check, making sure that each vertex handle on one surface skin has a matching vertex handle on the corresponding skin of an adjacent surface. |
Ok, is this reflected in the documention? |
Not yet. I'll add it into the doxygen documentation for check_mesh_for_watertightness. |
check_watertight, currently when no tolerance is provided defaults to zero, there should be either
The text was updated successfully, but these errors were encountered: