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

Download ahn produces only zeros with rasterio version 1.3.11 #369

Closed
rubencalje opened this issue Sep 17, 2024 · 6 comments
Closed

Download ahn produces only zeros with rasterio version 1.3.11 #369

rubencalje opened this issue Sep 17, 2024 · 6 comments

Comments

@rubencalje
Copy link
Collaborator

Showing warning:

WARNING:py.warnings._showwarnmsg:C:\Users\Ruben\miniconda3\envs\artesia\lib\site-packages\rasterio\merge.py:321: UserWarning: Ignoring nodata value. The nodata value, 3.4028234663852886e+38, cannot safely be represented in the chosen data type, float32. Consider overriding it using the --nodata option for better results. Falling back to first source's nodata value.

@dbrakenhoff
Copy link
Collaborator

Fixed by #370

@bdestombe
Copy link
Collaborator

bdestombe commented Sep 23, 2024

The ellipsis server used in the new method introduced in #370 seems to lack a few tiles. Some urls are nan's leading to errors further down the line. Those nan's are already in the json string retreived from the ellipsis server and does not seem to be an error on nlmod's side.
image

@bdestombe bdestombe reopened this Sep 23, 2024
@rubencalje
Copy link
Collaborator Author

Nice catch.

I think it is just the tile in the northsea, that has been renamed between version 2 and 3 of ahn, I guess. See the location of tiles 72Hz2 and 18HZ2 below. I will add a check that ignores nan-values in the tile-names. This is probably also needed for AHN 5 probably, which is not defined everywhere in the Netherlands jet.

18HZ2
72HZ2

@bdestombe
Copy link
Collaborator

Ah it took me some time to figure out, but the tiling is different depending on which server you use (legacy or ellipse).

The legacy function is currently not working. Ill commit a fix to solve both the legacy and the new function.

@bdestombe
Copy link
Collaborator

#371

@bdestombe
Copy link
Collaborator

Closed with #371

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

No branches or pull requests

3 participants