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 to match neurodatascience/nipoppy:main #67

Merged
merged 1 commit into from
Sep 29, 2023
Merged

Conversation

michellewang
Copy link
Collaborator

PR created/updated automatically by create-pull-request GitHub action. Will be merged automatically by automerge-action if there are no merge conflicts.

* verified shell check will work with planned data

* wip - updated pybids parsing to use filter and only map requested subject. fixed parsing / printing errors

* small update to parsing

* updated tractoflow run script to scilus-flows_v1.5.0 (hopefully)

* remove singularity call from nextflow b/c nextflow is in the container now

* (hopefully) working version of updated tractoflow

* removed submodule - no longer required w/ newer version

* intial attempt to update tracker for updated tractoflow

* *sigh* thank you codespell

* added sample_bids_filter.json; modidifed how bids_filter is called; simplified tracker; redirected nextflow logs

* updated tractoflow output directory to match other non-BIDS outputs (freesurfer in fmriprep)

* moved the bids_filter_tractoflow.json location to generalize for pipelines
Copy link
Collaborator

@bcmcpher bcmcpher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR approved by auto-approve-action GitHub action.

@michellewang michellewang merged commit 0ffe15d into main Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants