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

Update examples #62

Open
wants to merge 2 commits into
base: update-examples
Choose a base branch
from

Conversation

m-wierzba
Copy link
Contributor

@m-wierzba m-wierzba commented Nov 23, 2024

The previous PR was accidentally merged using push --force. This one continues the work started in the previous one.

This is needed when one works with a clone and wants the clone to have
the special remote readily available.
@mslw
Copy link
Contributor

mslw commented Dec 6, 2024

I looked at the current state and it looks pretty much as I would imagine it. There is very little, if anything, that would be missing from the narrative.

We spoke off the list and it seems that the main thing missing now is the fMRIPreped dataset to be cloned, which would provide a remake target (in general it would be good to have one, so that the tutorial can be completed without having to go through a full fMRIPrep run first). And this is the only reason for the PR being kept in the draft state. Whoever gets to it first can set it up; I am personally stuck with a provisioning problem for prospective execution (#69 - it seems noone else has experienced it before) and will prioritize understanding what happens there.

@m-wierzba m-wierzba marked this pull request as ready for review December 6, 2024 11:44
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.

2 participants