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

#843 - Added devcontainers #845

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

pvcasillasg
Copy link

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.

@pvcasillasg
Copy link
Author

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

@mj-will
Copy link
Collaborator

mj-will commented Nov 1, 2024

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.

@mj-will mj-will self-requested a review November 1, 2024 10:03
@@ -0,0 +1,8 @@
FROM continuumio/anaconda3

RUN conda install -c conda-forge bilby gwpy python-lalsimulation pymultinest
Copy link
Collaborator

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.

Copy link
Author

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.

@mj-will
Copy link
Collaborator

mj-will commented Nov 1, 2024

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

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

@mj-will mj-will linked an issue Nov 1, 2024 that may be closed by this pull request
@pvcasillasg
Copy link
Author

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add devcontainer file
2 participants