Skip to content

Update PyChaste Docker Image#52

Merged
kwabenantim merged 10 commits intodevelopfrom
docker
Jan 9, 2024
Merged

Update PyChaste Docker Image#52
kwabenantim merged 10 commits intodevelopfrom
docker

Conversation

@kwabenantim
Copy link
Copy Markdown
Member

Summary
Create an updated docker image for PyChaste.

Related Issues

@kwabenantim kwabenantim linked an issue Nov 28, 2023 that may be closed by this pull request
2 tasks
@kwabenantim
Copy link
Copy Markdown
Member Author

Building the image seems to have dependency resolution issues although pychaste conda packages have been built for Python 3.11 here: https://anaconda.org/PyChaste/repo.

Pinned packages:
  - python 3.11.*
  - python 3.11.6
  - jupyterlab >=4.0.4
  - notebook >=7.0.2
  
...
│  │  │  ├─ vtk 7.1.1 would require
│  │  │  │  ├─ jpeg >=9e,<10a , which can be installed;
│  │  │  │  ├─ libtiff >=4.5.0,<4.6.0a0  with the potential options
│  │  │  │  │  ├─ libtiff [4.4.0|4.5.0] would require
│  │  │  │  │  │  └─ libdeflate >=1.14,<1.15.0a0 , which can be installed;
│  │  │  │  │  ├─ libtiff 4.5.0 would require
│  │  │  │  │  │  └─ libdeflate >=1.17,<1.18.0a0 , which can be installed;
│  │  │  │  │  ├─ libtiff 4.5.0 would require
│  │  │  │  │  │  └─ libdeflate >=1.16,<1.17.0a0 , which can be installed;
│  │  │  │  │  └─ libtiff [4.5.0|4.5.1] would require
│  │  │  │  │     └─ libdeflate >=1.18,<1.19.0a0 , which conflicts with any installable versions previously reported;
│  │  │  │  └─ libxml2 >=2.10.3,<2.11.0a0  but there are no viable options
│  │  │  │     ├─ libxml2 2.10.3 would require
│  │  │  │     │  └─ icu >=70.1,<71.0a0 , which conflicts with any installable versions previously reported;
│  │  │  │     └─ libxml2 [2.10.3|2.10.4] would require
│  │  │  │        └─ icu >=72.1,<73.0a0 , which conflicts with any installable versions previously reported;
...

@kwabenantim
Copy link
Copy Markdown
Member Author

Building the image seems to have dependency resolution issues although pychaste conda packages have been built for Python 3.11 here: https://anaconda.org/PyChaste/repo.

Switched to micromamba image to start from a blank environment 804c3372.

@kwabenantim
Copy link
Copy Markdown
Member Author

The new image seems to be working now. Currently testing:
docker run --init -it --rm -p 8888:8888 kwabenantim/pychaste

@kwabenantim kwabenantim marked this pull request as ready for review January 9, 2024 09:29
@kwabenantim kwabenantim requested a review from mirams January 9, 2024 10:50
Copy link
Copy Markdown
Member

@mirams mirams left a comment

Choose a reason for hiding this comment

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

Haven't tried it out, but all changes look sensible!

@kwabenantim kwabenantim merged commit 0d6ad1e into develop Jan 9, 2024
@kwabenantim kwabenantim deleted the docker branch January 9, 2024 11:25
@bdevans
Copy link
Copy Markdown
Member

bdevans commented Jan 9, 2024

Slightly late to the party but it looks good! The main change I would consider is using the newer jupyterlab package rather than notebook (unless that was what caused the dependency issues above).

@kwabenantim
Copy link
Copy Markdown
Member Author

Slightly late to the party but it looks good! The main change I would consider is using the newer jupyterlab package rather than notebook (unless that was what caused the dependency issues above).

Thanks @bdevans. I'll make the changes in a new PR.

@bdevans
Copy link
Copy Markdown
Member

bdevans commented Jan 9, 2024

Oh, some labels are also a good idea. e.g. you could adapt these ones from the base image:

LABEL maintainer="Ben Evans <ben.d.evans@gmail.com>" \
    author.orcid="https://orcid.org/0000-0002-1734-6070" \
    image.publication="https://doi.org/10.21105/joss.01848" \
    org.opencontainers.image.authors="Benjamin D. Evans" \
    org.opencontainers.image.url="https://github.com/Chaste/chaste-docker" \
    org.opencontainers.image.licenses="MIT" \
    org.opencontainers.image.title="Chaste Docker Image" \
    org.opencontainers.image.description="Chaste: Cancer, Heart and Soft Tissue Environment" \
    org.opencontainers.image.documentation="https://chaste.github.io/docs/installguides/docker/"

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.

Update PyChaste docker image

3 participants