-
Notifications
You must be signed in to change notification settings - Fork 5
[WIP] Update Ubuntu base and Cuda #20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
@mattcieslak how can I get it to try building using this new Dockerfile? |
|
you have to push a tag for the build to happen. Also, wdyt about aiming for a lower cuda? I think the penn cluster uses 12.2 |
|
Ah ok. I think 12.2 should be fine. In fact, I think its not just caused by the CUDA major version per se, just that its an older dockerfile and many of the surrounding libraries are out of date. |
cbda0e4 to
c47e611
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Bug: Ubuntu Upgrade Breaks Qt PPA Integration
The Ubuntu base image upgrade to 22.04 breaks the Qt PPA installation. The ppa:beineri/opt-qt-5.12.8-bionic repository is specific to Ubuntu 18.04 (bionic) and won't work with Ubuntu 22.04 (jammy), causing the build to fail when attempting to add the repository or install packages from it.
Dockerfile#L196-L197
Lines 196 to 197 in c47e611
| # Install qt5.12.8 | |
| RUN add-apt-repository ppa:beineri/opt-qt-5.12.8-bionic \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
8af471b to
c949b12
Compare
Updates:
These changes necessitated updating other packages, so I took the liberty to update:
This may necessitate concomitant changes in qsiprep
This is first steps in resolving this issue:
PennLINC/qsiprep#935
Note
Bumps all images to Ubuntu 22.04 with CUDA 12.2.2, migrates to Qt6, updates DSI Studio and key neuroimaging stacks (ANTs, TORTOISE), and refreshes the conda environment and build tags.
ubuntu:22.04withCUDA 12.2.2.jammyand updatecmake.qt6-base-devandlibqt6charts6-dev./opt/dsi-studio-cpu; add toPATHand strip Qt ABI tags for Singularity.ANTS 2.4.3, switch to Makefiles, and update toolchain; copy pruned binaries.libboost-all-dev; refreshTORTOISE_SHA; rebuild against updated ITK/CMake.ubuntu:22.04and adjust runtime deps.qsiprep_env.yml: bumpnumpy/scipy, addvtk, updatedipy,dmri-amico,pyafq,scikit-*,trx-python, adddmri-dicelib,fury.ubuntu:22.04, pin license file to a specific commit, ensureinference.pyis executable.setup_build.shto25.11.14and fixTAG_TORTOISECUDAusage.Written by Cursor Bugbot for commit c949b12. This will update automatically on new commits. Configure here.