Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Change behavior of grid_x, y, and z for rectilinear grids #107

Merged
merged 1 commit into from
Apr 6, 2020

Conversation

mdpiper
Copy link
Member

@mdpiper mdpiper commented Feb 28, 2020

For a rectilinear grid, these functions should return, respectively, a vector of length number of columns, number of rows, and number of zlevels, not a vector of length node_count. The behavior of these functions is unchanged for other grid types.

See more on this at csdms/bmi#65.

For a rectilinear grid, these functions should return, respectively, a
vector of length ncols, nrows, and nzlevels.
@coveralls
Copy link

coveralls commented Feb 29, 2020

Coverage Status

Coverage decreased (-0.1%) to 71.358% when pulling b31f588 on mdpiper/modify-gridxyz into f8463fd on master.

@mdpiper
Copy link
Member Author

mdpiper commented Apr 2, 2020

@mcflugen Are you good with this change? If so, I'd to merge it & tag the result pymt=1.1.1; I need this chunk of code for the PRMS components.

@mcflugen
Copy link
Member

mcflugen commented Apr 6, 2020

@mdpiper 👍 Looks good.

@mdpiper mdpiper merged commit e80b2da into master Apr 6, 2020
@mdpiper mdpiper deleted the mdpiper/modify-gridxyz branch April 6, 2020 17:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants