-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add UKB ingression #147
Add UKB ingression #147
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #147 +/- ##
==========================================
- Coverage 33.53% 33.52% -0.01%
==========================================
Files 57 57
Lines 7117 7119 +2
Branches 965 967 +2
==========================================
Hits 2387 2387
- Misses 4628 4630 +2
Partials 102 102 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
What was the change in workbench? |
There wasn't an update to workbench specifically. The update from |
This helps account for duplicate subject names in the case multiple sessions of a subject are ingressed. Also accounts for if the BIDS ID does not match the original input folder name.
5e12e88
to
d4084dc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM too
at least for DSI Studio recon-specs
Changes proposed in this pull request
Update I2Q
Documentation that should be reviewed
Just the input data section of the docs.