Skip to content
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

docker and singularity images for aydin (gpu) #59

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

truatpasteurdotfr
Copy link

Hi,

Initial version if you wish to use it, more a proof of concept than production ready!

  • miniconda3 based recipe
  • caveat: miniconda:master for debian11, may need to switch to miniconda:latest when upstream switch
  • cudatoolkit version 11.3 from miniconda requires Nvidia driver recent enough (11.3 required driver >=465.19.01)
  • docker image does not have ENTRYPOINT (I don't use docker, bbut ymmv)
  • works for me but a test case should be provided

Cheers

Tru

@AhmetCanSolak
Copy link
Collaborator

hello @truatpasteurdotfr !

Pardon my late reply, I was out for some days.
I will give this a try and review it this week but looks good at first sight!
Thank you for the PR.

@truatpasteurdotfr
Copy link
Author

I have splitted the CI for docker and singularity containers as I am hitting disk size limit on the CI runner.

@royerloic royerloic requested a review from AhmetCanSolak May 18, 2022 14:51
@AhmetCanSolak
Copy link
Collaborator

Hi @truatpasteurdotfr ,

We are close to finish our migration to pytorch from tensorflow/keras. I know this been a long waiting PR but after completion of that migration, we can update this branch again and merge it.

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