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

xyz_to_cid optional z #259

Open
tomvansteijn opened this issue Sep 8, 2023 · 0 comments
Open

xyz_to_cid optional z #259

tomvansteijn opened this issue Sep 8, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@tomvansteijn
Copy link

Currently there are two functions mapping x,y-coordinates to model coordinates: xyz_to_cid mapping x, y and z to layer, row, column on a structured grid or icell2d on a vertex grid, and xy_to_icell2d mapping x and y to icell2d on a vertex grid. I think it would be easier if to have one function with an optional z argument, allowing the user to enter the layer separately.

@dbrakenhoff dbrakenhoff added enhancement New feature or request good first issue Good for newcomers labels Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants