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

Faces and centers #73

Open
ocefpaf opened this issue Jan 18, 2016 · 6 comments
Open

Faces and centers #73

ocefpaf opened this issue Jan 18, 2016 · 6 comments
Labels

Comments

@ocefpaf
Copy link
Member

ocefpaf commented Jan 18, 2016

@ayan-usgs, when browsing the namespace one can easily find everything that is related to nodes with sgrid.node<tab> (sgrid.node_coordinates, sgrid.node_dimensions, sgrid.node_lon, sgrid.node_lat, and sgrid.node_padding). However, when doing the same for the center we only find sgrid.center_lon and sgrid.center_lat.

Does it make sense to rename the sgrid.face_{coordinate, dimensions, padding} to sgrid.center_{coordinate, dimensions, padding}? Note that sgrid.faces is always none!

@ayan-usgs
Copy link
Contributor

@ocefpaf,

Yep, doing the rename makes sense.

@ocefpaf
Copy link
Member Author

ocefpaf commented Jan 19, 2016

Thanks @ayan-usgs!

FYI I am experimenting a new API and ways to make all slices and computations lazy:

http://nbviewer.jupyter.org/gist/ocefpaf/28d9279479022454bb99

It is not ideal yet because of a masking issue. If I can figure that out things will be much easier for sci-wms.

@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 9, 2016

Can you cross- reference the PR that did the rename? I am a bit lost.

@ayan-usgs
Copy link
Contributor

The rename happened in #72. The variables were named face_coordinates, face_dimensions, etc.

@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 9, 2016

I center_{lon,lat} are still there. The question was regarding re-naming those.

@ocefpaf ocefpaf reopened this Feb 9, 2016
@ayan-usgs
Copy link
Contributor

Sorry. Didn't totally re-read the question... 😭

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

No branches or pull requests

2 participants