Skip to content

add notebook that uses titiler-pgstac endpoints for raster mosaics#423

Merged
hrodmn merged 4 commits intodevelopfrom
hr/icsat2-boreal-mosaic
Sep 20, 2024
Merged

add notebook that uses titiler-pgstac endpoints for raster mosaics#423
hrodmn merged 4 commits intodevelopfrom
hr/icsat2-boreal-mosaic

Conversation

@hrodmn
Copy link
Contributor

@hrodmn hrodmn commented Sep 5, 2024

  • use the icesat2-boreal dataset as an example

  • demonstrate the /collections and /searches endpoints

The tile performance is pretty slow for the icesat2-boreal dataset - maybe there are some database tweaks that could improve performance or maybe just a larger database instance would help!

* use the icesat2-boreal dataset as an example

* demonstrate the `/collections` and `/searches` endpoints
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@hrodmn hrodmn requested a review from wildintellect September 5, 2024 21:23
@hrodmn hrodmn self-assigned this Sep 5, 2024
Copy link

@hanbyul-here hanbyul-here left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👋 I was trying to run this notebook locally and got the error that featureBounds is not defined. I worked around it by using the bounds information from stac, but leaving the comment here in case this helps!

@wildintellect
Copy link
Collaborator

@hanbyul-here thanks for catching that, I hadn't run the notebook yet. @hrodmn are you missing an import for the featureBounds function?

@hrodmn
Copy link
Contributor Author

hrodmn commented Sep 6, 2024

Yeah, I found the issue - will clean it up and push a change!

@hrodmn
Copy link
Contributor Author

hrodmn commented Sep 16, 2024

@wildintellect I addressed all of your comments - anything else you want to change before we merge this one? @hanbyul-here is there anything we should add based on feedback from the presentation at FOSS4GNA?

@hanbyul-here
Copy link

Nothing from my end! (and I could run the entire notebook!)

@wildintellect
Copy link
Collaborator

wildintellect commented Sep 20, 2024

@hrodmn I think all that's missing is adding the page to the table of contents. https://github.com/MAAP-Project/maap-documentation/blob/develop/docs/source/technical_tutorials.rst

sorry the visualization sub-index https://github.com/MAAP-Project/maap-documentation/blob/develop/docs/source/technical_tutorials/visualizing.rst

@hrodmn hrodmn merged commit 7b4a96d into develop Sep 20, 2024
@hrodmn hrodmn deleted the hr/icsat2-boreal-mosaic branch September 20, 2024 18:18
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.

4 participants