Skip to content

Conversation

@ayenpure
Copy link
Collaborator

@ayenpure ayenpure commented Nov 4, 2025

  • Earlier the connectivity and data files might have different dim names for the horizontal axis. This change considers the first dimension in the connectivity file as the horizontal axis and then matches corresponding data file dimension name based on arrity.
  • This change still assumes that lev/ilev are the chief vertical dimensions. Hence the variables are still split as 1D/2D/3Di/3Dm. Eventually based on the land model considerations we will need to omit this catagorization and present all variables with the dimension information.
  • Currently, the app also lacks support for dynamic selection for slicing along an arbitrary dimension -- a key change necessary to present all variables in atmosphere and land models. A strategy to implement in ParaView first and then in QuickView is needed.

  - Earlier the connectivity and data files might have different dim
    names for the horizontal axis. This change considers the first
    dimension in the connectivity file as the horizontal axis and then
    matches corresponding data file dimension name based on arrity.
  - This change still assumes that lev/ilev are the chief vertical
    dimensions. Hence the variables are still split as 1D/2D/3Di/3Dm.
    Eventually based on the land model considerations we will need to
    omit this catagorization and present all variables with the
    dimension information.
  - Currently, the app also lacks support for dynamic selection for
    slicing along an arbitrary dimension -- a key change necessary to
    present all variables in atmosphere and land models. A strategy to
    implement in ParaView first and then in QuickView is needed.
@ayenpure
Copy link
Collaborator Author

ayenpure commented Nov 4, 2025

@huiwanpnnl This addresses only the naming discrepancy problem; for fixing other issues, Berk and I will continue to discuss a good strategy for both ParaView and QuickView.

@ayenpure ayenpure merged commit d75c5e4 into master Nov 4, 2025
1 check passed
@ayenpure ayenpure deleted the dimension_matching branch November 12, 2025 23:36
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

Successfully merging this pull request may close these issues.

1 participant