Skip to content

Conversation

@bcmcpher
Copy link
Contributor

@bcmcpher bcmcpher commented May 1, 2025

Closes #4

Working version of files from SOBP hackathon.

@bcmcpher bcmcpher changed the title Iss4 qsirecon Add qsirecon May 1, 2025
@michellewang
Copy link
Contributor

@bcmcpher could you add ('qsirecon', '1.1.0', 'default') here?

@bcmcpher
Copy link
Contributor Author

bcmcpher commented May 6, 2025

Added. Expect a merge conflict w/ the freewater branch - same line added in that PR

Copy link
Contributor

@michellewang michellewang left a comment

Choose a reason for hiding this comment

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

🧑‍🍳

@michellewang
Copy link
Contributor

@bcmcpher do you want to add an HPC config file too? See #10

@michellewang michellewang changed the title Add qsirecon Add qsirecon 1.1.0 Jul 17, 2025
Copy link
Contributor

@michellewang michellewang left a comment

Choose a reason for hiding this comment

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

Codespell fails because there are typos in the descriptor but since they are typos from the pipeline developers we decided to leave them as they are

@michellewang michellewang merged commit b56b8a5 into neurodatascience:main Jul 24, 2025
2 of 3 checks passed
michellewang added a commit that referenced this pull request Jul 25, 2025
* initial version of qsirecon

* dropped zenodo.json config

* added pipeline to test file

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* added default hpc.json

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: Michelle Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
michellewang added a commit that referenced this pull request Aug 14, 2025
* update descriptor for bids-validator

* add test

* update test

* ignore VS code settings

* update descriptors and config files if needed

* add workaround for bosh failing with dcm2bids 3.2.0 and heudiconv

* Add qsirecon 1.1.0 (#8)

* initial version of qsirecon

* dropped zenodo.json config

* added pipeline to test file

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* added default hpc.json

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: Michelle Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Add fwDTI 2.0.0 (#7)

* added files w/ pipeline name

* added pipeline to test file

* updated w/ zenodo config

* fixed name, dropped old

* Update tests/conftest.py

add new version

Co-authored-by: Michelle Wang <[email protected]>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* dropped unused, previous versions; adjusted config

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* ignore qsirecon typos

* address PR review comments

* fix test

* remove globbing from invocation

---------

Co-authored-by: Michelle Wang <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* update descriptor for dmri-freewater pipeline

* update qsirecon

* [BUG] Improve tests and fix bug in QSIRecon config (#22)

* update tests to make them install the pipelines

* fix qsirecon config missing variables

* update tracker tests

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* flake8

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

---------

Co-authored-by: Brent McPherson <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

Add QSIRecon latest version

2 participants