-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Support Meshtag IO for more types (only int32_t
is supported at present)
#1253
Comments
I do not see any. Just |
Simplicity. Is there a compelling reason/need to support Is there a current use case for double? |
I was currently planning to use the double one as a workaround for visualizing DG 0 functions, as we are lacking this functionality. |
int32_t
is supported at present)
@jorgensd do we have this now? |
@garth-wells we do not |
Is this still needed? It would be fairly easy to add more types (which? just |
We should at least support float/double, as some use-cases read material parameters from the mesh . Ref: https://fenicsproject.discourse.group/t/i-o-from-xdmf-hdf5-files-in-dolfin-x/3122/48?u=dokken Where scalar data is actually a meshtag double object. |
We should add support for more types, but need to clean up the HDF5 interface first to better handle different types. |
The meshtag IO is limited to int32, while the meshtags support
int8, int32, int64 and double. Is there a particular reason for this?
The text was updated successfully, but these errors were encountered: