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

Duplicate nodal data in Tecplot surface files #366

Open
lamkina opened this issue Nov 8, 2024 · 0 comments
Open

Duplicate nodal data in Tecplot surface files #366

lamkina opened this issue Nov 8, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@lamkina
Copy link
Contributor

lamkina commented Nov 8, 2024

Description

The Tecplot writer is duplicating the nodal data of the entire surface mesh for every zone in the Tecplot file. This doesn't show up when opening the file in Tecplot because it's indexed by the connectivity, which contains the correct number of elements and indexing for the global node array.

This duplicated nodal data is bloating our Tecplot surface files on the order of the number of mesh zones times the total number of nodes in the surface mesh.

Ideally, we should only write the nodes needed for each zone, or use a shared data array when writing the file. The connectivity may have to be updated as well depending on the chosen solution strategy.

Steps to reproduce issue

Use the Tecplot reader from this PR: mdolab/baseclasses#95

Print the length of the nodal data stored in each variable for each zone to see they are all the size of the full surface mesh.

Plotting a scatter of the nodal data will show that all nodes are stored for every zone.

@lamkina lamkina added the bug Something isn't working label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant