khanlab / scattr

BIDS app for structural connectivity between targeted regions
https://scattr.readthedocs.io/en/stable/
MIT License
4 stars 3 forks source link

Docker #42

Closed kaitj closed 1 year ago

kaitj commented 1 year ago

Proposed changes

This updates the Docker file so everything is contained within the singularity image (allowing it to be directly run from the container), with the container itself being around 6GB.

Should be ready for a review of the code, but needs to be tested on actual data prior to merging. Would also like to hold off on merging this so that a 0.1.0 version can first be released (basically everything that was used as part of the OHBM abstract) Also planning to, in a future PR, update all of the containers to make use of this new container.

Resolves #23 and #41.

One thing I would like to do here is to merge this, and before creating the next release update the config with the next docker version so that the workflow actually uses the self-contained with updated dependencies.

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

kaitj commented 1 year ago

Would propose holding this off until #44 and #43 are merged in, and this is rebased following the name change. This would be more of a v0.1.1 release.

kaitj commented 1 year ago

@tkkuehn Mind taking another look at this when you get a chance? The Dockerfile was updated and there were some workflow updates as a result as well.