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

Data grabbing #1

Open
effigies opened this issue Sep 4, 2018 · 5 comments
Open

Data grabbing #1

effigies opened this issue Sep 4, 2018 · 5 comments

Comments

@effigies
Copy link
Member

effigies commented Sep 4, 2018

Brain extraction relies on a template. In niworkflows we have data-grabbing facilities, which I can incorporate here, but it may be worth splitting these out into their own package (nipreps/niworkflows#250). Part of the reason for this is that niworkflows logging and paths are baked into that code, so if we're going to spend time generalizing, we might as well do it once, and do it right.

@satra
Copy link
Collaborator

satra commented Sep 4, 2018

is this something that could be replaced using the datalad python api? coupled with the potential thought of a datalad interface for nipype itself.

@effigies
Copy link
Member Author

effigies commented Sep 4, 2018

Does datalad provide an interface to tarballs on OSF? If so, maybe. The data grabbing stuff in niworkflows does two things: hash-validates and unzips data packages, and checks data directories/caches that can be defined at the system and user level.

@satra
Copy link
Collaborator

satra commented Sep 4, 2018

datalad can index tarballs - not very efficient, because almost the entire tar file has to be downloaded to extract a file. osf has individual file level URLs as well. the latest iteration of git-annex has a notion of a global cache, otherwise it's always a local cache. but when you say defined at the system/user level. that can easily be done in current git-annex.

@effigies
Copy link
Member Author

effigies commented Sep 4, 2018

Okay. We may need to upload a non-tarball variant of the OASIS template, but that sounds reasonable to me.

@effigies
Copy link
Member Author

effigies commented Oct 2, 2018

At least for the time being, I've just included the needed files directly. I think this should be usable, but I need to test on a system that has ANTs installed. Or get to the point of building an image in Circle.

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

No branches or pull requests

2 participants