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

bump the docker image versions for the Terra notebooks #470

Merged
merged 3 commits into from
Sep 29, 2021

Conversation

amygdala
Copy link
Collaborator

... plus typo fix

Copy link
Collaborator

@deflaux deflaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and thank you for the typo fix!

Can you please also update the Docker tag in the rest of the notebooks that use image ghcr.io/broadinstitute/ml4h/ml4h_terra?

@deflaux
Copy link
Collaborator

deflaux commented Sep 29, 2021

This change also fixes #456

@amygdala
Copy link
Collaborator Author

Sorry, I spaced that the review_results ones needed updating. PTAL. (did I miss anything else?) @deflaux

Copy link
Collaborator

@deflaux deflaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lg, thanks @amygdala !

@amygdala amygdala merged commit 3dc52bc into broadinstitute:master Sep 29, 2021
@amygdala amygdala deleted the terra_nb_image_vs branch September 29, 2021 19:26
lucidtronix pushed a commit that referenced this pull request Jan 13, 2023
* bump the docker image versions for the Terra notebooks
* typo fix
lucidtronix pushed a commit that referenced this pull request Jan 13, 2023
* bump the docker image versions for the Terra notebooks
* typo fix
lucidtronix pushed a commit that referenced this pull request Jan 13, 2023
* bump the docker image versions for the Terra notebooks
* typo fix
lucidtronix pushed a commit that referenced this pull request Jan 13, 2023
* bump the docker image versions for the Terra notebooks
* typo fix
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