Skip to content

Conversation

@tsalo
Copy link
Contributor

@tsalo tsalo commented Feb 25, 2025

Addresses #926 (comment).

@tsalo tsalo requested a review from mgxd February 25, 2025 18:00
Copy link
Contributor

@mgxd mgxd left a comment

Choose a reason for hiding this comment

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

im not sure why i included // in #857 ...wdyt about removing that and just mirroring your changes to the other entities?

@tsalo
Copy link
Contributor Author

tsalo commented Feb 25, 2025

Sounds good. I'll do that now.

@codecov
Copy link

codecov bot commented Feb 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.40%. Comparing base (72327c5) to head (f014788).
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #927   +/-   ##
=======================================
  Coverage   69.40%   69.40%           
=======================================
  Files          86       86           
  Lines        8483     8483           
  Branches     1059     1059           
=======================================
  Hits         5888     5888           
  Misses       2373     2373           
  Partials      222      222           

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

@effigies effigies merged commit f412c8e into nipreps:master Feb 25, 2025
22 checks passed
effigies added a commit that referenced this pull request Mar 20, 2025
1.13.0 (March 20, 2025)

New feature release in the 1.13.x series.

This release restructures the ANTs brain extraction workflow.
This should be a backwards compatible change, but a minor release is used out of caution.

* FIX: Remove duplicated code, problematic deviation from antsBrainExtraction (#931)
* FIX: Listify numbers (#925)
* ENH: Allow + symbol in label entities (#926, #927)
* STY: Apply various ruff checks (#913, #915, #917, #918, #919, #930)
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.

3 participants