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

Replacing legacy vtk file writing with h5py file writing for meshes, geometry and tracks for paraview > 5.13.0 #3113

Open
shimwell opened this issue Aug 9, 2024 · 0 comments

Comments

@shimwell
Copy link
Member

shimwell commented Aug 9, 2024

Description

I'm attending at a Paraview presentation by @mwestphal and heard about this nice new feature that I think we should make use of in openmc.

Paraview is supporting hdf5 files if written in a specific way with h5py 🎉 .

Here is the relevant blog post https://www.kitware.com/how-to-write-time-dependent-data-in-vtkhdf-files/ with a nice example of writing the data using h5py

This is interesting for me because:

  • We currently have a vtk dependency which could be removed
  • We already have h5py as a dependency which can be utilized more
  • The vtk files we currently write are legacy format and not recommend by paraview anymore
  • The hdf5 files we could write for our 3d data to would be smaller in size and quicker to load than the .vtk files

The parts of the code that could be updated are

Alternatives

Stick with the current solutions, or add an arg to allow "legacy" vtk file writing

Compatibility

Will require ParaView 5.13.0 or newer to open the hd5f based files

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

No branches or pull requests

1 participant