-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
PostGIS Raster integration #112
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
fa6a6c4
PostGIS Raster integration
48a6530
PostGIS Raster integration: creating postgis_raster extension in tests
a869134
PostGIS Raster integration: explicit conversion of records to return …
b9acb57
PostGIS Raster integration: "CREATE OR REPLACE AGGREGATE" removed for…
6e0f7dc
PostGIS Raster integration: documentation generation: "RETURNS TABLE"…
ee3373c
PostGIS Raster integration: type creation fix
3d00489
PostGIS Raster integration: raster geometry transform fix
423470f
PostGIS Raster integration: rewritten to re-use raster polygon
cc81a6a
PostGIS Raster integration: minor documentation update
a95b8ce
PostGIS Raster integration: redundant subqueries removed
b7e2fb4
PostGIS Raster integration: documentation update
ff8bd85
PostGIS Raster integration: method selection based on pixels per cell…
de88f47
PostGIS Raster integration: STRICT removed for helper functions that …
51142e7
PostGIS Raster integration: ST_Buffer "join=mitre" parameter added fo…
7c753f7
PostGIS Raster integration: cell geometry filtering fix
4f4bfb3
PostGIS Raster integration: method selection condition adjusted
42625ac
PostGIS Raster integration: always using hexagons for cell area estim…
18ea23b
PostGIS Raster integration: documentation update
711d80c
PostGIS Raster integration: pixel area estimation fix
1c29e8b
PostGIS Raster integration: documentation update
9a08ab7
PostGIS Raster integration: documentation update
94c4dfc
PostGIS Raster integration: geometry to geography conversion fix
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
This query frightens me as documentation, likely because it lacks sample output to grasp what it does.
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.
Added sample output and a description.
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.
Interested and innocent bystander: It would be good to have an example of this but where the return value is simpler, e.g. just the value at the H3 cell centre, or the return value is the mode value, rather than these complex objects.
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.
@alpha-beta-soup
Thank you, that's a good idea. I've added an example of getting the most frequent value in each cell.
The other example shows an intended use case, and I also tried to show how to calculate a total # of pixels in each cell, since it's not immediately obvious.