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

VTKExporter #828

Open
KnutAM opened this issue Oct 14, 2023 · 4 comments
Open

VTKExporter #828

KnutAM opened this issue Oct 14, 2023 · 4 comments

Comments

@KnutAM
Copy link
Member

KnutAM commented Oct 14, 2023

As suggested by @fredrikekre on Slack during the discussions about #692, this issue contains the notes for one potential way this could be implemented on top of #692

Having a type that can cache the transformation to the vtk data structures makes sense to avoid having to redo this each time step when using a VTKFileCollection.

Furthermore, this could then be customized to contain the special mappings required for e.g. discontinuous interpolations (xref) and user customizations (perhaps for, e.g., IGA/Embedded, @lijas?).

For example, we could have

struct VTKFile{VTK<:WriteVTK.VTKFile, VTKExp<:VTKExporter}
    vtk::VTK
    vtk_exporter::VTKExp
end
@termi-official
Copy link
Member

Related to this we might also want to check out if we want to do some kine of VTKImporter, which can e.g. help for post-processingin Julia.

@fredrikekre
Copy link
Member

Done in #692.

@KnutAM
Copy link
Member Author

KnutAM commented Jun 30, 2024

Some developments towards this in #867 for one case, but I would keep this open to remind that it might make sense to have the buffer general there (to support also other types of discontinuous interpolations / cases. (But to be done after 1.0!)

@KnutAM KnutAM reopened this Jun 30, 2024
@KnutAM
Copy link
Member Author

KnutAM commented Jul 1, 2024

See also lijas/IGA.jl#11, seems like having a "vtk_exporter" field would allow re-using VTKFile there, and dispatch some methods on VTKFile{<:Any, IGAExporter}, such that users could call VTKFile in both cases (Maybe type-params in VTKFile should be switched to simplify this...

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

No branches or pull requests

3 participants