[go: up one dir, main page]

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

Remove Fides file output #3212

Open
garth-wells opened this issue May 14, 2024 · 5 comments
Open

Remove Fides file output #3212

garth-wells opened this issue May 14, 2024 · 5 comments
Labels
io proposal Suggested change or addition question Further information is requested
Milestone

Comments

@garth-wells
Copy link
Member

Fides doesn't add anything over other formats (less, in fact), is more code to maintain and I doubt it is used.

Should we remove it?

@jorgensd

@garth-wells garth-wells added question Further information is requested proposal Suggested change or addition labels May 14, 2024
@jorgensd
Copy link
Member

I've not used it for a very long time, let's keep it until we have a PR for VTKHDF, and then remove it (and let other users chime in).

@francesco-ballarin
Copy link
Member

I am in favor of removing fides file output as soon as possible. Ping @massimiliano-leoni who may have an opinion on this topic.

@mleoni-pf
Copy link
Contributor

Thanks for pinging me!

I am mainly using VTX and Fides nowadays, the former for high-order elements and the latter for first-order elements. I could admittedly use VTX for everything, Fides is much faster to read in Paraview but that's not really our concern. Does VTX support all function types and elements that Fides supports?

@jorgensd
Copy link
Member

Thanks for pinging me!

I am mainly using VTX and Fides nowadays, the former for high-order elements and the latter for first-order elements. I could admittedly use VTX for everything, Fides is much faster to read in Paraview but that's not really our concern. Does VTX support all function types and elements that Fides supports?

I think so yes. Fides is limited to first order, so it doesn’t really yield much.

@mleoni-pf
Copy link
Contributor

In that case I will just use VTX in the future.

@garth-wells garth-wells added the io label Jun 1, 2024
@garth-wells garth-wells added this to the 0.10.0 milestone Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
io proposal Suggested change or addition question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants