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

Added new data for distributed closest point tests. #11

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

gunney1
Copy link
Contributor

@gunney1 gunney1 commented Oct 12, 2022

Added small 2D, multidomain meshes for testing distributed closest point search. The search code now supports domain over- and underloading by using multidomain meshes.

The script for generating simple multidomain meshes will be added to Axom but will not be used in test recipes because it requires a conduit library configured with python3 and hdf5.

Copy link
Member

@agcapps agcapps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Brian.

Copy link
Member

@rhornung67 rhornung67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gunney1 why can't we test with HDF5 and python3? Our TPL build of conduit is built with HDF5 and python3 is supported on LC platforms.

@gunney1 gunney1 merged commit f5b8fe7 into main Oct 12, 2022
@gunney1
Copy link
Contributor Author

gunney1 commented Oct 12, 2022

Thanks @gunney1 why can't we test with HDF5 and python3? Our TPL build of conduit is built with HDF5 and python3 is supported on LC platforms.

@rhornung67 We can. But the conduit build itself has to be configured to support its python interface. /usr/WS1/axom/libs/toss_3_x86_64_ib/2022_07_05_19_44_26/clang-9.0.0/conduit-0.8.3, which is what the host config for rzgenie uses, wasn't configured with python3.

Moreover, I'm not sure we want to run the mesh generation script as part of the axom test. It's more of an exercise in using conduit.

@gunney1 gunney1 deleted the feature/gunney/multidomain-closest-point-search branch October 12, 2022 23:12
@kennyweiss
Copy link
Member

@rhornung67 -- I suspect that using Conduit's python interface requires configuring conduit as a shared library.
We can add a host-config with that option (and might have had one in the past(?)) but I don't think we currently have that configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants