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

Add Container Definitions #10

Open
wants to merge 3 commits into
base: SABR_ready
Choose a base branch
from
Open

Conversation

kkaulen
Copy link

@kkaulen kkaulen commented Apr 15, 2024

As MN-BaB strongly relies on C dependencies, getting it to run, especially on compute clusters where one does not have root access, can be quite a hustle. Therefore, I added Docker and Apptainer (formerly Singularity) Image definitions to give users an easy way to set MN-BaB up, on personal computers as well as compute clusters.

I added documentation in the README file to get both images to run.

The available CUDA images are based on Ubuntu 20.04, so I updated MN-BaB to work with Python 3.8. This was more straightforward than getting Python 3.7 to run in the images. This led to a change in the network loading code (not sure why it was needed to reference the blocks property of the network earlier) and in the requirements.txt (I also fixed all dependencies and versions to make the build process bullet-proof).

I hope the maintainers see this as a valuable addition to make MN-BaB runnable by a broader audience of researchers. I would be very happy to address any questions you might have and to work on any further suggestions!

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.

None yet

1 participant