Skip to content

Conversation

tbarsballe
Copy link
Contributor

@tbarsballe tbarsballe commented Sep 16, 2025

This MR updates the Dockerfile to use jupyter/minimal-notebook:8515ad39f045, and includes additional dependency changes to support this:

  • Adjust the libgl packages to ones that exist on modern ubuntu
  • Adjust the requirements to include cartopy, which must be installed by conda and is required by the later pip install of descartes geoviews
  • Remove the legacy nbextensions, as ipyleaflet and ipywidgets should now be supported by Jupyter 7 without additional configuration required.

TODO:

  • Test opencv notebook to verify new libgl install
  • Test descartes/geoviews notebook to verify updated cartopy install
  • Test leaflet notebook to verify ipyleaflet extension works as expected under Jupyter 7
  • [ ] Update documentation to match new Jupyter interface, where applicable. Everything still works as before, no doc updates necessary

@tbarsballe
Copy link
Contributor Author

Verified jupyter-notebooks/use-cases/coastal-erosion-example/4_masks_and_filters.ipynb works with the new Dockerfile - OpenCV appears to still be working.

@tbarsballe
Copy link
Contributor Author

tbarsballe commented Sep 16, 2025

Verified descartes and ipyleaflet using jupyter-notebooks/Analytics-API/user-guide
/01_getting_started_with_the_planet_analytics_api.ipynb
, though we do get a (correct) warning:

[/tmp/ipykernel_13168/1689031255.py:2](http://localhost:8888/tmp/ipykernel_13168/1689031255.py#line=1): UserWarning: Geometry is in a geographic CRS. Results from 'centroid' are likely incorrect. Use 'GeoSeries.to_crs()' to re-project geometries to a projected CRS before this operation.

Might be worth adding some config to the notebook to mute that warning

@tbarsballe
Copy link
Contributor Author

Tested geoviews on jupyter-notebooks/Analytics-API/user-guide/03_change_detection.ipynb

geoviews looks like it's still working fine, but that particular notebook is pretty out-of-date - none of the feed ids in the notebook exist anymore, and many of the properties referenced have been removed or renamed in modern change detection feeds. I'm not up-to-speed enough on Analytics API or change detection for fix those issues.

@tbarsballe tbarsballe changed the title Draft: Update Dockerfile to 2025 build of jupyter/minimal-notebook Update Dockerfile to 2025 build of jupyter/minimal-notebook Sep 16, 2025
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.

1 participant