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

Creating pvtu file #38

Open
kursatyurt opened this issue Oct 29, 2021 · 3 comments
Open

Creating pvtu file #38

kursatyurt opened this issue Oct 29, 2021 · 3 comments

Comments

@kursatyurt
Copy link
Collaborator

I am trying to add parallel output in case we run in parallel. VTK has native support for parallel vtu output using class vtkXMLPUnstructuredGridWriter here which automatically creates master .pvtu file and .vtu files. (Naming convention is not same as preCICE case.pvtu and ranks are case_<rank>.vtu) . However, to use this class in a parallel case vtkMPIcontroller is required and it only comes when VTK compiled with MPI support Ubuntu and Fedora repos have without MPI. Probably using this method creates lots of issues (installing VTK from source with MPI support) on the user side, and reduces usability.

The other way of doing a parallel output is writing individual meshes using the vtkXMLUnstructuredGridWriter class and creating a master file manually. It seems a bit tricky since we don't have headers of .vtu files, is there a proper way to create a master file in this case?

If the expected output is the same as the one from preCICE do we really need an output routine? Can't we directly use the output of preCICE and remove output from ASTE (yes it is a bit tricky but if we can use one functionality from a tool why do we need to reimplement it again)?

@fsimonis
Copy link
Member

fsimonis commented Nov 4, 2021

Naming convention is not same as preCICE

Actually, we could call this a bug and fix it on develop. I'll open an issue.

The other way of doing a parallel output is writing individual meshes using the vtkXMLUnstructuredGridWriter class and creating a master file manually. It seems a bit tricky since we don't have headers of .vtu files, is there a proper way to create a master file in this case?

ASTE is so simple that you can easily hardcode the main file. Checkout the VTKXMLExporter.cpp in precice.

If the expected output is the same as the one from preCICE do we really need an output routine? Can't we directly use the output of preCICE and remove output from ASTE (yes it is a bit tricky but if we can use one functionality from a tool why do we need to reimplement it again)?

I see your point, however, the output of the tool has to integrate with the rest of the tooling independent of the configuration. This simplifies name handling etc.
Letting ASTE write the results is the easiest way to achieve that.

@davidscn
Copy link
Member

davidscn commented Dec 2, 2021

@kursatyurt is this still relevant now that we have agreed on a proper naming convention in preCICE?

@kursatyurt
Copy link
Collaborator Author

@kursatyurt is this still relevant now that we have agreed on a proper naming convention in preCICE?

Right now naming convention is the same, but we are still not creating any pvtu master file in ASTE.

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

3 participants