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

Add lonboard and upgrade stac_ipyleaflet #40

Open
2 tasks
abarciauskas-bgse opened this issue Feb 2, 2024 · 6 comments
Open
2 tasks

Add lonboard and upgrade stac_ipyleaflet #40

abarciauskas-bgse opened this issue Feb 2, 2024 · 6 comments

Comments

@abarciauskas-bgse
Copy link
Contributor

We would like to add lonboard to the default VEDA JupyterHub image. There are a few cases for lonboard in VEDA, one which should be a notebook NASA-IMPACT/veda-docs#115, but another would be to visualize DEMs generated by lidar (a use case I heard @naomatheus is working on.

@wildintellect mentioned we should also include an upgrade of stac_ipyleaflet in this issue as well, as we would like to see if lonboard could work on top of stac_ipyleaflet.

Acceptance criteria:

  • lonboard comes installed in the default VEDA JupyterHub image.
  • Visualizing results from a stac query with lonboard is tested and works as expected.
@wildintellect
Copy link
Collaborator

To be clear stac_ipyleaflet has a major bugfix, we need to bump to 0.3.6 to make it work again https://github.com/MAAP-Project/stac_ipyleaflet/releases/tag/0.3.6
@sandrahoang686 @ranchodeluxe @emmalu do we have notes for the process to release on VEDA jupyterhub?

@ranchodeluxe
Copy link
Collaborator

ranchodeluxe commented Feb 5, 2024

We have documentation on this process that most folks have walked through already. What we don't have documentation on is a full upgrade (including base image) that this ticket (#41) is attempting to do. But that checklist pretty accurate enough to start it

@abarciauskas-bgse
Copy link
Contributor Author

While testing notebooks for #43 I upgraded stac_ipyleaflet to 0.3.6 as mentioned above but also had to add a cell with the following to get a working stac_ipyleaflet:

import os 
os.environ['TITILER_STAC_ENDPOINT']='https://staging-raster.delta-backend.com/stac'
os.environ['TITILER_ENDPOINT']='https://staging-raster.delta-backend.com/'
os.environ['STAC_CATALOG_NAME']='VEDA STAC'
os.environ['STAC_CATALOG_URL']='https://staging-stac.delta-backend.com/'

should I go ahead and add that to the docs as-is or wait until stac_ipyleaflet is upgraded?

@wildintellect
Copy link
Collaborator

@abarciauskas-bgse that info belongs here, users should not have to know about that, it's supposed to ship with the VEDA deployment of stac_ipyleaflet cc: @emmalu @sandrahoang686
We'll make sure to include those parameters in the image build with the upgrade.

@emmalu
Copy link
Collaborator

emmalu commented Feb 15, 2024

Those environment variables should already be included, they were first released in v0.3.3 which is installed in the main (first) VEDA Jupyter env:
Screenshot 2024-02-15 at 6 13 05 PM
400c77de5d1209a515ac228ff752fa1513d2c85c

I did test upgrading up to v0.3.6 within the notebook and (after restarting the kernel), the OS variables seem to have persisted... 🤔
Screenshot 2024-02-15 at 6 26 47 PM

@weiji14
Copy link
Member

weiji14 commented Apr 2, 2024

FYI, there's some interest in adding anywidget/lonboard to pangeo-docker-images at pangeo-data/pangeo-docker-images#520. I'm adding anywidget for now at pangeo-data/pangeo-docker-images#522, and lonboard could probably be added somewhere here later:

dependencies:
- ipykernel
- pip
- pip:
- git+https://github.com/MAAP-Project/stac_ipyleaflet.git@0.3.3#egg-info=stac_ipyleaflet

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

No branches or pull requests

5 participants