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.

Sign up for GitHub

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

Jump to bottom

Option to load minimal required coordinates and connectivity #954

Open
rajeeja opened this issue Sep 13, 2024 · 4 comments · May be fixed by #1092 or #1101
Open

Option to load minimal required coordinates and connectivity #954

rajeeja opened this issue Sep 13, 2024 · 4 comments · May be fixed by #1092 or #1101
Assignees
@philipc2 @aaronzedwick
Labels
new feature New feature or request

Comments

@rajeeja
Copy link
Contributor

rajeeja commented Sep 13, 2024

Proposed new feature or change:

This "minimal" reader would only reads corner nodes and face node connectivity, ignoring other info in grid file. #953
@rajeeja rajeeja added the new feature New feature or request label Sep 13, 2024
@github-project-automation github-project-automation bot moved this to 📚 Backlog in UXarray Development Sep 13, 2024
@philipc2
Copy link
Member

This could be generalized for all other grid formats also.

@philipc2 philipc2 added this to the MPAS milestone Sep 15, 2024
@aaronzedwick
Copy link
Member

I can tackle this, when you say corner nodes do you mean just read in the nodes of the grid? @rajeeja

@aaronzedwick aaronzedwick self-assigned this Oct 1, 2024
@rajeeja
Copy link
Contributor Author

rajeeja commented Oct 1, 2024

I can tackle this, when you say corner nodes do you mean just read in the nodes of the grid? @rajeeja

💯 Just be careful, sometimes there will be center nodes, edge nodes also. How to tackle those?

@aaronzedwick
Copy link
Member

It shouldn't be too hard, I figure we could even use our current readers, but pass in a minimal tag or something, which tells it to only read the face_x/y/z/lat/lon and face_node_connectivity

@erogluorhan erogluorhan moved this from 📚 Backlog to 📝 To-Do in UXarray Development Nov 26, 2024
@philipc2 philipc2 changed the title Minimal reader for MPAS etc. Option to load minimal required coordinates and connectivity Dec 3, 2024
@philipc2 philipc2 self-assigned this Dec 3, 2024
@philipc2 philipc2 removed this from the MPAS milestone Dec 3, 2024
@philipc2 philipc2 moved this from 📝 To-Do to 🏗 In progress in UXarray Development Dec 3, 2024
@philipc2 philipc2 linked a pull request Dec 3, 2024 that will close this issue
DRAFT: Better use Xarray for Loading Grids #1092
Draft
This was referenced Dec 6, 2024
Draft
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philipc2 philipc2

@aaronzedwick aaronzedwick

Labels
new feature New feature or request
Projects
UXarray Development
Status: 🏗 In progress
Milestone
No milestone
3 participants
@rajeeja @philipc2 @aaronzedwick