Skip to content

TST: Test comprehensively dMRI b0 volume handling#364

Merged
jhlegarreta merged 1 commit into
nipreps:mainfrom
jhlegarreta:tst/test-handling-b0-dwi-post-init
Nov 29, 2025
Merged

TST: Test comprehensively dMRI b0 volume handling#364
jhlegarreta merged 1 commit into
nipreps:mainfrom
jhlegarreta:tst/test-handling-b0-dwi-post-init

Conversation

@jhlegarreta
Copy link
Copy Markdown
Contributor

Test comprehensively dMRI b0 volume handling:

  • Add a test that checks the behavior of the DWI class __attrs_post_init__ method that handles setting the bzero attribute and masking the dataobj and gradient data corresponding to the b=0 volumes.
  • Further parametrize the post init error handling function to check the behavior of the __attrs_post_init__ method in a broader range of values. Specifically, check that the initialization fails when the number of provided volumes is right at the limit of the number of the required DTI orientations.

@jhlegarreta jhlegarreta force-pushed the tst/test-handling-b0-dwi-post-init branch 3 times, most recently from c9fac88 to cfa5b5c Compare November 29, 2025 16:27
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.95%. Comparing base (301429b) to head (6829199).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   80.95%   80.95%           
=======================================
  Files          34       34           
  Lines        1959     1959           
  Branches      207      207           
=======================================
  Hits         1586     1586           
  Misses        321      321           
  Partials       52       52           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Test comprehensively dMRI b0 volume handling:
- Add a test that checks the behavior of the DWI class
  `__attrs_post_init__` method that handles setting the `bzero`
  attribute and masking the `dataobj` and gradient data corresponding to
  the b=0 volumes.
- Further parametrize the post init error handling function to check the
  behavior of the `__attrs_post_init__` method in a broader range of
  values. Specifically, check that the initialization fails when the
  number of provided volumes is right at the limit of the number of the
  required DTI orientations.
@jhlegarreta jhlegarreta force-pushed the tst/test-handling-b0-dwi-post-init branch from cfa5b5c to 6829199 Compare November 29, 2025 16:41
@jhlegarreta jhlegarreta merged commit ec545eb into nipreps:main Nov 29, 2025
8 checks passed
@jhlegarreta jhlegarreta deleted the tst/test-handling-b0-dwi-post-init branch November 29, 2025 16:53
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.

1 participant