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

GeoTiff float (Digitales Höhenmodell Oberfläche NRW) #274

Open
pfefferCode opened this issue Feb 12, 2024 · 8 comments
Open

GeoTiff float (Digitales Höhenmodell Oberfläche NRW) #274

pfefferCode opened this issue Feb 12, 2024 · 8 comments

Comments

@pfefferCode
Copy link

Hi!

The problem occurs using tiffs like this: https://www.wcs.nrw.de/geobasis/wcs_nw_dom?REQUEST=GetCoverage&SERVICE=WCS&VERSION=2.0.1&COVERAGEID=nw_dom&FORMAT=image/tiff&SUBSET=x(356000,357000)&SUBSET=y(5699000,5700000)&OUTFILE=dom1_32_356_5699_1_nw_2020&APP=timonline

(hint you can pick your own .tiff by: this URL: https://www.tim-online.nrw.de/tim-online2/?bg=basemapDE_grau&scale=1400000&center=450000,5700000&wms=https://www.wms.nrw.de/geobasis/wms_nw_dhm-uebersicht,nw_dhm_metadaten&opacity=70&legend=true (klick on "download" an choose "Höhenmodelle" --> "Digitales Oberflächenmodell" --> "weiter" --> "*.tiff" --> "weiter" --> select one --> "weiter"....)

I tried it on http://app.geotiff.io/

There are two issues that might be connected:

  1. Loading the .tif file is working always only at the second try. After the first try the area is marked all in the same gray. After the second loading of the image, correct grey values are shown.
  2. Identifying values on http://app.geotiff.io/ does not give a result. In the JavaScript console appears the following error:
bundle.js:35 Uncaught TypeError: Cannot read properties of null (reading 'then')
    at r.<anonymous> (bundle.js:35:1059117)
    at r.fire (bundle.js:92:5274)
    at r._fireDOMEvent (bundle.js:92:43297)
    at r._handleDOMEvent (bundle.js:92:42540)
    at HTMLDivElement.a (bundle.js:92:22968)
    at r.<anonymous> (bundle.js:35:1059117)
    at r.fire (bundle.js:92:5274)
    at r._fireDOMEvent (bundle.js:92:43297)
    at r._handleDOMEvent (bundle.js:92:42540)
    at HTMLDivElement.a (bundle.js:92:22968)
(anonymous) @ bundle.js:35
fire @ bundle.js:92
_fireDOMEvent @ bundle.js:92
_handleDOMEvent @ bundle.js:92
a @ bundle.js:92
(anonymous) @ bundle.js:35
fire @ bundle.js:92
_fireDOMEvent @ bundle.js:92
_handleDOMEvent @ bundle.js:92
a @ bundle.js:92

Thank you for your work!
PfefferCode.

PS: I first reported it here: geotiffjs/geotiff.js#419 but I was told, here is the correct place.

@DanielJDufour
Copy link
Member

DanielJDufour commented Feb 13, 2024

This should be fixed with dcf846e#diff-094f1c9b3122487b86e34893f4c4138beeb0af18744c773aac10e4cc30c3c55e.

Lemme know if it works for you!

@DanielJDufour
Copy link
Member

Also, would you be interested in submitting this file to https://github.com/geotiff/test-data ? I don't think we have other test files in the EPSG:25832 projection. We'll also be able to use it to test the visualization once it's added.

@pfefferCode
Copy link
Author

pfefferCode commented Feb 13, 2024

@DanielJDufour Thanx for answering and provoding a fix so fast!
How can I test if it works? (When) will it be included in http://app.geotiff.io/ ?

Please feel free to add the .tif to the test data.

@DanielJDufour
Copy link
Member

It should be included in app.geotiff.io right now :-)

Note: the identify function should be working, but the visualization is a work in progress.

@pfefferCode
Copy link
Author

@DanielJDufour
Thanx! identifying works now :-)
Showing the values as different grey tones still work only on the second load of the same image.
All other functions (Mean, Sum, Mode, Median, Histogram - did not test Band arithmetic and Raster calculator) do not work.

@DanielJDufour
Copy link
Member

Okay. I'll fix the other stats now. Thanks for testing!

@DanielJDufour
Copy link
Member

DanielJDufour commented Feb 23, 2024

Mean, Min, Max, Sum, Mode, and Median should all be fixed now. Will have to look into Histogram, Band Arithmetic, and Raster Calculator.

@pfefferCode
Copy link
Author

tested: mean, min, max, sum, mode an median as rectangle and polygon: all deliver plausible values. Thank you.
Histogram does not work till now.

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

2 participants