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
[WIP] Add Freesurfer container #23
base: main
Are you sure you want to change the base?
Conversation
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.
Pretty much everything is there ! Once the comments are fixed, it should be good. You don't need scilpy from what I see ?
docker-bake.hcl
Outdated
@@ -116,6 +116,10 @@ variable "SCILUS_TAG" { | |||
variable "FLOWS_TAG" { | |||
} | |||
|
|||
variable "FS_VERSION" { |
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.
variable "FS_VERSION" { | |
variable "freesurfer-version" { |
Also move the variable definition higher, with the other dependencies (names are lowercase)
ARG FREESURFER_VERSION | ||
|
||
# shell settings | ||
WORKDIR /root |
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.
WORKDIR /root | |
WORKDIR / |
docker-bake.hcl
Outdated
target = "freesurfer-install" | ||
contexts = { | ||
freesurfer-base = "docker-image://${base-install-image}" | ||
freesurfer-builder = "docker-image://${base-build-image}" |
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.
Remove, you don't have a build stage
docker-bake.hcl
Outdated
freesurfer-builder = "docker-image://${base-build-image}" | ||
} | ||
args = { | ||
FREESURFER_VERSION = "${fs-version}" |
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.
FREESURFER_VERSION = "${fs-version}" | |
FREESURFER_VERSION = "${freesurfer-version}" |
docker-bake.hcl
Outdated
"type=registry,ref=${dockerhub-user-pull}/build-cache:scilus-deps", | ||
"type=registry,ref=scilus/build-cache:scilus-deps" | ||
] | ||
tags = notequal("", DEPS_TAG) ? stamp_tag("scilus/scilus-deps:${DEPS_TAG}", timestamp()) : [] |
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.
tags = notequal("", DEPS_TAG) ? stamp_tag("scilus/scilus-deps:${DEPS_TAG}", timestamp()) : [] | |
tags = notequal("", DEPS_TAG) ? stamp_tag("scilus/scilus-freesurfer:${DEPS_TAG}", timestamp()) : [] |
let's put this in a separate dockerhub repository.
ENV MNI_PERL5LIB /usr/local/freesurfer/mni/share/perl5 | ||
ENV PERL5LIB /usr/local/freesurfer/mni/share/perl5 | ||
|
||
ENV FS_LICENSE='/license.txt' |
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.
Add the version to the common VERSIONS file, like the other dockerfiles.
No description provided.