Conversation
Added thsi line: python3 -m pip install --no-cache-dir --break-system-packages v${GENOMAD_VER}.tar.gz && \
|
This PR supersedes #1517 |
|
The tests worked |
|
I don't think this is an error, but it may concern users. I think this error is from tensorflow, and might be silence-able by telling it upfront that there's no GPU or CUDA in this image. What happens if you set the environmental variable It'd be something like |
ENV PATH="/mmseqs/bin:$PATH" LC_ALL=C CUDA_VISIBLE_DEVICES=""
|
Okay I just added the update you suggested to the dockerfile. It appears to be working without using Cuda/GPUs now. Build Output |
|
It looks like the error is still there |
|
Hmmm I ran it again with |
|
@erinyoung I think we missed this error before, and it exists in the previous image. Replacing @cwoodside1278, Could you try installing |
|
@erinyoung If it works without problem, I think you can approve as it is 🙈 |
I updated the code to contain tensorflow-cpu, but I also left the old code just commented out in case we need it or need to replicate the bug
|
Okay I made that update to include tensorflow-cpu and my docker build was fine once again. Can you let me know if the issue is still happening? @erinyoung @Kincekara |
|
It looks like the error message is gone! Huzzah! |
erinyoung
left a comment
There was a problem hiding this comment.
I'll get this merged and deployed.
Thank you for putting this together!
|
The status of the deploy can be found here : https://github.com/StaPH-B/docker-builds/actions/runs/22648851692 |
Pull Request (PR) checklist:
docker build --tag samtools:1.15test --target test docker-builds/build-files/samtools/1.15)docker-builds/build-files/spades/3.12.0/Dockerfile)build-files/shigatyper/2.0.1/test.sh)docker-builds/build-files/spades/3.12.0/README.md)Description
This PR is for Genomad update to 1.11.2 from 1.11.1
Diff output
Test Build