-
Notifications
You must be signed in to change notification settings - Fork 2
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
Overlap checker tool #29
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI looks good. All set to be merged after the very minor changes requested here.
Fixed the inconsistent indentations you highlighted and a couple more I noticed, lmk if anything else needs editing. I also removed the parsing of an argument to hold the number of points along a triangle edge to sample since I don't think we will be implementing that feature anymore and will instead go straight to ray_fire along edges check. |
@Waqar-ukaea it looks like CI is still unhappy. Did the submodule get updated since that PR to the test files repo was merged? |
Added overlap checker tool from DAGMC to check and identify overlapped regions in .h5m faceted files.
XDG::MeshManager
does not have support to call MOAB functions that return entity adjacenciesTested the tool on some h5m files locally and it seems like its working okay. It identifies when there are overlaps and which volumes are overlapped in agreement with DAGMC's version. Also added a new test file that runs the 5 test cases that DAGMC does for its overlap-checker tool.
One thing to note, the overlap location printed when running the tool doesn't necessarily match DAGMC's. I'm not sure what the cause of that is or if that is a major concern? Maybe due to differences in the order in which vertices are checked between the libraries?
Edit - CI fails because the .h5m files needed for those tests are missing from the repository. The files to be used with overlap-check's test's can be found here - test_files.zip. The CMakeLists.txt expects these files to be placed in
${XDG_ROOT}/tests/test_files