Open
Conversation
- Based on official EOPF image (registry.eopf.copernicus.eu/cpm/eopf-cpm:2-6-3) - Installs JupyterHub, JupyterLab with extensions - Includes Dask and Dask Gateway support - Compatible with JupyterHub deployments Resolves #43
- Lists all required JupyterHub and Dask packages - Provides alternative installation method - Documents dependencies for the image Related to #43
- Overview and features - Building and running instructions - JupyterHub configuration examples - Docker Hub repository information Related to #43
- Add PYTHONPATH environment variable to make packages installed in /home/dask/.local/ accessible to all users - Ensures xarray, zarr, netCDF4, and dask_gateway from base image are available when running JupyterLab or JupyterHub - Update main README with eopf-jupyterhub-dask image information
Contributor
Author
|
@senmao @clausmichele requesting your review. |
Member
clausmichele
left a comment
clausmichele
left a comment
There was a problem hiding this comment.
In general it's fine. I'm already testing the image with the help of @senmao. Once we are able to fix this EOPF-Sample-Service/eopf-sample-notebooks#186 (comment) we can merge this PR.
Comment on lines
+65
to
+70
| ## Related Issues and PRs | ||
|
|
||
| - Issue: [#43 - Create Docker image with eopf compatible with JupyterHub and Dask Cluster](https://github.com/EOPF-Sample-Service/eopf-container-images/issues/43) | ||
| - Related Issue: https://github.com/EOPF-Sample-Service/eopf-sample-notebooks/issues/185 | ||
| - Related PR: https://github.com/EOPF-Sample-Service/eopf-sample-notebooks/pull/186 | ||
|
|
Member
There was a problem hiding this comment.
I would remove this part, not relevant for the README
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #43 - Docker image for SAFE to Zarr conversion with JupyterHub and Dask