Run resection identification using Docker container#920
Run resection identification using Docker container#920chinmaychinara91 wants to merge 12 commits into
Conversation
Extend `RunContainer` to accept isGpu and containerArgs parameters (with defaults) so callers can request GPU access and pass additional docker arguments
Useful when the MATLAB function errors or there is some user interrupt (e.g. Ctrl+C)
When GetEngine searches for engine binaries, prepend common macOS Docker locations to PATH (/usr/local/bin, ~/.docker/bin, /Applications/Docker.app/Contents/Resources/bin) so the subsequent 'which' call can locate Docker-related executables. This improves engine detection on macOS systems.
chinmaychinara91
left a comment
There was a problem hiding this comment.
@rcassani I have tested these for Windows and Mac (Apple Silicon) and it works fine. Would need you help on testing it on Linux.
|
@chinmaychinara91, did you notice any strange behaviour on using the Specifically, in some tests @Edouard2laire reported the image SHA returned by FYI @Edouard2laire |
It works fine as it is now on Apple Silicon. On pulling, the correct image SHA is obtained. Nothing strange was observed there. |
|
@Edouard2laire was it a specific scenario where you could not get the correct image SHA ? |
I just tried with the example image, and got this:
with the following plugin: But the image is downloaded, just with a different sha: docker pull returns this:
I am using Matlab 2023b; Here are the info of docker:
|
|
It seems, on my side that the digest is different form the imageID: It seems to be discussed here: https://stackoverflow.com/questions/56364643/whats-the-difference-between-a-docker-images-image-id-and-its-digest With docker image ls --digests , we can get the link between the two ids: |
|
@rcassani just wanted to gently follow up on the next steps for this PR. Kindly let me know how you would like to proceed. |
|
|
@rcassani I resolved the requested changes. One thing I noticed: this error message appears in the command window: It looks like
Sounds good. I will update the tutorial as well once you give a go ahead. |






This PR introduces the execution of the
resection-identificationplugin by transitioning it to run within a Docker container, utilizing the recent container-based plugin support in Brainstorm (#899).Related forum discussion: https://neuroimage.usc.edu/forums/t/automatic-resection-identification/56589/16
Usage
Anatomy > resection-identificationplugin in Brainstorm to load the image and start the container.Checklist
FYI: @tmedani @ajoshiusc @Edouard2laire