Skip to content

Conversation

@jhlegarreta
Copy link
Contributor

Test instantiating the dMRI models through the factory.

Test instantiating the dMRI models through the factory.
@jhlegarreta jhlegarreta force-pushed the tst/test-dmri-factory-init branch from 27c0769 to e93906b Compare October 1, 2025 23:47
@codecov
Copy link

codecov bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.97%. Comparing base (431ce51) to head (e93906b).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #252      +/-   ##
==========================================
+ Coverage   77.55%   77.97%   +0.41%     
==========================================
  Files          24       24              
  Lines        1439     1439              
  Branches      166      166              
==========================================
+ Hits         1116     1122       +6     
+ Misses        252      247       -5     
+ Partials       71       70       -1     

☔ 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.

@jhlegarreta
Copy link
Contributor Author

If the PET models should also be allowed to be instantiated through the factory (related PR #250) the iterator, this will be added and modified in a separate PR as the factory will require fixes. Merging.

@jhlegarreta jhlegarreta merged commit cb3a2da into nipreps:main Oct 2, 2025
9 of 10 checks passed
@jhlegarreta jhlegarreta deleted the tst/test-dmri-factory-init branch October 2, 2025 00:05
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