Skip to content

Conversation

plule-ansys
Copy link
Contributor

  • Update jupyter-hub to its latest version and switch to the ":hub-" base-notebook tag: Jupyter lab will complain on version difference between jupyterlab and the base images
  • Update all the conda/pip/labextension dependencies
  • Reenable the cadquery notebook that was fixed by these updates. Still need the :master notation for cadquery though.
  • Move to ghcr.io and simplify the CI accordingly (image name = repo name)

@@ -1,6 +1,6 @@
FROM jupyter/base-notebook:python-3.8.8
FROM jupyter/base-notebook:hub-1.4.2
Copy link
Contributor

Choose a reason for hiding this comment

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

Guess 3.8 isn't the latest anyway. Between the python-3.9 and hub images, I'm not seeing any difference in the docker layers, so either one works.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@akaszynski ah yep I did this change because I had warnings of version difference between the jupyter hub base image and the version of the helm chart, so I thought tagging by hub version was more significant here

Copy link
Contributor

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

LGTM.

We might need to make some tweaks in regards to remote visualization with pythreejs, but that's still experimental and we can do that later.

Co-authored-by: Alex Kaszynski <akascap@gmail.com>
@plule-ansys plule-ansys merged commit 9d080fa into master Oct 21, 2021
@akaszynski akaszynski deleted the update_jupyter branch December 18, 2021 03:44
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.

2 participants