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

fix: add order by to make values be picked up #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

merydian
Copy link

The querys ST_Value returns a table with one col and multiple rows. A row for each raster tile that ST_Intersects returns true for. Previously, all except one value at tile borders were null. The valid value was at the bottom of the table returned by ST_Value. Only the topmost point is picked up by the application. The order by now makes sure that the valid value is at the top of the table in order to be picked up.

ST_Value returns a table with one col and multiple rows. A row for each raster tile that ST_Intersects  returns true for. Previously, all except one value at tile borders were null. The valid value was at the bottom of the table returned by ST_Value. Only the topmost point is picked up by the application.
The order by now makes sure that valid value is at the top of the table in order to be picked up.
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

Successfully merging this pull request may close these issues.

1 participant