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

Simple use case examples to resolve #373 #395

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jakepalmer
Copy link

I've put together some example use cases as suggested in #373 by @dbkeator. I've included a single T1 nifti (my own defaced scan so no problem with permissions/privacy etc) to enable users to simple clone and copy/paste commands from the README. I can re-work if inclusion of data like this isn't ideal for any reason.

@kaczmarj
Copy link
Collaborator

this is really great -- thank you @jakepalmer.

i've been inactive here for a while, but i'm back now. we have a website now at https://www.repronim.org/neurodocker/ and this would fit really well there. could you please add these things to docs/ -- wherever you think they make sense.

@djarecka
Copy link
Member

@jakepalmer - thank you for your PR and sorry for late reply.

@kaczmarj @satra - I found this old PR that adds additional use cases for working with data. This PR adds also scripts to docs, so I'm not sure if we want to merge it in this form. Any thoughts?

@kaczmarj
Copy link
Collaborator

instead of storing the data and scripts in this github repo, i would prefer to include a download command to get sample nifti files, and to maybe paste the scripts into the common_uses.rst file itself. otherwise this looks great.

@stebo85
Copy link
Collaborator

stebo85 commented Aug 22, 2023

agree with @kaczmarj that the example file shouldn't be in the github repository.

@jakepalmer could you change this example to a download link and update the pull request?

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.

4 participants