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

Work on functions for eoapi #56

Merged
merged 2 commits into from
Mar 15, 2023
Merged

Work on functions for eoapi #56

merged 2 commits into from
Mar 15, 2023

Conversation

bitner
Copy link
Contributor

@bitner bitner commented Feb 20, 2023

PLAYGROUND TO SEE WHAT WORKS

pg_temp.pgstac_collections_view should work just like a normal collection

pg_temp.pgstac_hash takes a queryhash as the argument and then will use the "automagic" bounds geometry calculated from the z/x/y

http://localhost:8083/collections/pg_temp.pgstac_hash/tiles/0/0/0?queryhash=c53f4ebe9b2a50bc2b52fd88a5d503e1 is an example with registering search('{}') as the query in pgstac.

If only someone hadn't gotten rid of the vector tile viewer, we'd be able to see the MVT results ;-P

@bitner
Copy link
Contributor Author

bitner commented Feb 20, 2023

@vincentsarago

@vincentsarago
Copy link
Member

vincentsarago commented Feb 21, 2023

pg_temp.pgstac_hash works great 🥳 but I'm getting projection/geometries issue with the collection function

Screenshot 2023-02-21 at 7 08 02 PM

@bitner
Copy link
Contributor Author

bitner commented Mar 15, 2023

@vincentsarago do you have any stac definitions and urls that I can use to replicate the issues you are seeing with the collections view? At least with the one stac record for noaa-emergency-response it seems to work correctly

@bitner
Copy link
Contributor Author

bitner commented Mar 15, 2023

@vincentsarago

This comment was marked as resolved.

@vincentsarago
Copy link
Member

Thanks @bitner 🙏 everything seems to work.

I'll merge to the other PR and continue the work

@vincentsarago vincentsarago marked this pull request as ready for review March 15, 2023 20:26
@vincentsarago vincentsarago merged commit 12da835 into tiPg Mar 15, 2023
@vincentsarago vincentsarago deleted the tipg-functests branch March 15, 2023 20:26
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.

2 participants