Skip to content

FIX: Handle sessions (repeats) appropriately in matching algorithm#331

Merged
DESm1th merged 2 commits intoTIGRLab:masterfrom
jerdra:fix/bidsify-use-session
Jun 3, 2022
Merged

FIX: Handle sessions (repeats) appropriately in matching algorithm#331
DESm1th merged 2 commits intoTIGRLab:masterfrom
jerdra:fix/bidsify-use-session

Conversation

@jerdra
Copy link
Copy Markdown
Contributor

@jerdra jerdra commented Jun 3, 2022

Previously bidsify was using just the series number but this is an incorrect assumption when repeats are included. This fix adds a filter to the list of potential candidate scans by additionally matching on the session number.

@auto-assign auto-assign bot requested review from benselby, josephmje and jskocic June 3, 2022 14:40
@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Jun 3, 2022

Hello @jerdra, Thank you for updating!

Cheers! There are no style issues detected in this Pull Request. 🍻 To test for issues locally, pip install flake8 and then run flake8 datman.

Comment last updated at 2022-06-03 14:42:32 UTC

@jerdra jerdra requested review from DESm1th and removed request for jskocic June 3, 2022 14:41
@DESm1th DESm1th merged commit 790c6e6 into TIGRLab:master Jun 3, 2022
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