-
Notifications
You must be signed in to change notification settings - Fork 17
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
Hotfix/update-raster #162
Hotfix/update-raster #162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@krishnaglodha changes requested. Also, the exercise text (https://github.com/geopython/diving-into-pygeoapi/blob/main/workshop/content/docs/publishing/ogcapi-coverages.md) needs to be updated to reflect the new coverage data sample.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When review comments from @tomkralidis resolved GTG!
Looks good now. For my understanding: I run the exercise but apart from the data extent I don't see any raster data rendering from the GeoTIFF in the demo page: http://localhost:5000/collections/tartu-ntl . Or is that not intended (same I see now with Firenze data from 2022)? |
@justb4 correct, We are not rendering coverages. I will do a final review and merge later today. |
Updated coverages to work with local Tartu NTL data