NASA-IMPACT / veda-jh-environments

1 stars 1 forks source link

Add lonboard and upgrade stac_ipyleaflet #40

Open abarciauskas-bgse opened 10 months ago

abarciauskas-bgse commented 10 months ago

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 https://github.com/NASA-IMPACT/veda-docs/issues/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:

wildintellect commented 10 months ago

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 commented 9 months ago

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 (https://github.com/NASA-IMPACT/veda-jh-environments/issues/41) is attempting to do. But that checklist pretty accurate enough to start it

abarciauskas-bgse commented 9 months ago

While testing notebooks for https://github.com/NASA-IMPACT/veda-jh-environments/pull/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 commented 9 months ago

@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 commented 9 months ago

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 commented 8 months ago

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

https://github.com/NASA-IMPACT/veda-jh-environments/blob/2f72add4e605508f9b6a5d4bddf00df3e07e10cb/docker-images/custom/nasa-veda-singleuser/environment.yml#L4-L8