-
Notifications
You must be signed in to change notification settings - Fork 66
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
add support for mixed element unstructured meshes #1309
Conversation
@nicolemarsaglia Added mixed element support for both for bp -> vtkm and vtkm -> bp. Added tests that exercise round trip by using a pipeline to extract a subset of each mixed mesh. |
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.
Other than the todo checks you noted, I think it all looks great!
Is the mixed element portion of this PR in a state where it is worth testing/integrating into our code? |
@FreddieWitherden we are close with this. Aiming to get his merged next week. There is a windows specific issue and we want to add some more error checking logic. This will add support for mixed We will work on polytopal support later. |
Windows crashes due to an invalid memory access. Had bad luck getting a full debug stack built on windows,so I created an asan clang build: ASAN issue flagged:
|
vtk-m offsets expects an extra offsets entry that points to the end of the connectivity array. |
…pine-DAV/ascent into task/2025_05_mixed_ele_umeshs
found the source of the windows testing woes: LLNL/conduit#1297 adjusted the mixed tests to avoid the bug |
…pine-DAV/ascent into task/2025_05_mixed_ele_umeshs
resolved outstanding issue, merging. |
No description provided.