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

check elevation of highest pressure level in initialization #56

Open
2 tasks
ArcticSnow opened this issue Feb 22, 2023 · 1 comment
Open
2 tasks

check elevation of highest pressure level in initialization #56

ArcticSnow opened this issue Feb 22, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers new feature topoClass

Comments

@ArcticSnow
Copy link
Owner

ArcticSnow commented Feb 22, 2023

In the case the user misses pressure level high enough in the climate forcing dataset, the problem is identified only at the downscaling step. We can add a small check in the topoclass object initialization checking we have sufficient pressure levels.

  • check elevation of the highest pressure levl against highest DEM elevation (Zplevel > Zdem)
  • implement an ERROR statement suggesting which pressure level to get.

for task 1: check the $min(Z_{highest plevel} (x,y,time)) > max(Z_{dem}(x,y))$

@ArcticSnow ArcticSnow self-assigned this Feb 22, 2023
@ArcticSnow ArcticSnow added the good first issue Good for newcomers label Feb 22, 2023
@ealonsogzl
Copy link
Collaborator

I suppose that optionally the necessary pressure levels could even be obtained automatically from the DEM? It would avoid having to wait twice for the CDS queue if by mistake the right levels are not set.

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 new feature topoClass
Projects
None yet
Development

No branches or pull requests

2 participants