-
Notifications
You must be signed in to change notification settings - Fork 71
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
#843 - Added devcontainers #845
base: main
Are you sure you want to change the base?
Conversation
Oh, btw, since im using the continuumio/anaconda3 container image as base, it's a quite big image, it will be a great idea to create a registry for this, and just lay the build process on a pipeline, and modify the devcontainer.json |
Hi @pvcasillasg, thanks for making a start on this! I've added some initial comments but @ColmTalbot may have some more comments since he's managed the containers we use in the CI. |
@@ -0,0 +1,8 @@ | |||
FROM continuumio/anaconda3 | |||
|
|||
RUN conda install -c conda-forge bilby gwpy python-lalsimulation pymultinest |
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.
We may want to use a .yml file to configure the environments, what do you think?
Also, I think the gw_requirements.txt
file includes the minimum dependencies we need for the GW dockerfile, though we may wish to add more, e.g. pytest
and the other dependencies we use for testing.
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.
Seems a good solution, i will have a look over this today at afternoon.
Thanks about the context with gw_requirements, i will take that on mind, i will try to provide you a easy solution for add and modify dependencies in case it's necessary.
We have something along those lines implemented for the CI containers, see https://github.com/bilby-dev/bilby/blob/main/.github/workflows/build-containers.yml |
Oh, that's nice, i will try to develop a solution for upload the docker image on the registry. |
Hello!
I have created the devcontainers using the libraries from conda-forge, as indicated in your installation manual. If you want me to modify it to install in a different way, please let me know.
It’s also possible that some Python libraries may be missing.
I ran a couple of tests on the GW devcontainer, and the example supernova_example.py ran without any issues.