Skip to content

[FIX] SLF and callosal defaults#182

Merged
36000 merged 1 commit into
tractometry:mainfrom
36000:stronger_cleaning_before_release
May 8, 2026
Merged

[FIX] SLF and callosal defaults#182
36000 merged 1 commit into
tractometry:mainfrom
36000:stronger_cleaning_before_release

Conversation

@36000

@36000 36000 commented May 3, 2026

Copy link
Copy Markdown
Collaborator

Callosal Bundles returned to Mahalanobis cleaning, for now.

Stronger cleaning on SLF subbundles.

Both of these may need refinement before 3.0 ; Motor and Temporal bundles may need to be removed. They often are more incorrect compared to our other default bundles.

@36000 36000 added this to the pyAFQ 3.0 milestone May 3, 2026
@36000
36000 requested review from arokem and Copilot May 3, 2026 02:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts default cleaning behavior for SLF sub-bundles and callosal bundles, aligning bundle defaults with current segmentation/cleaning preferences ahead of the 3.0 release.

Changes:

  • Adds stricter Mahalanobis cleaning parameters applied to all SLF sub-bundles.
  • Removes Isolation Forest cleaning from callosal bundle defaults, reverting them to Mahalanobis-based cleaning.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AFQ/api/bundle_dict.py
Comment on lines +558 to +565
criteria_for_all={
"mahal": {
"distance_threshold": 3,
"length_threshold": 3,
"clean_rounds": 5,
"remove_lengths": "both",
},
},
@36000

36000 commented May 5, 2026

Copy link
Copy Markdown
Collaborator Author

TODO: try seeding in corpus callosum

@36000 36000 changed the title [WIP/FIX] SLF and callosal defaults [FIX] SLF and callosal defaults May 8, 2026
@36000
36000 force-pushed the stronger_cleaning_before_release branch from c543661 to d50ed12 Compare May 8, 2026 21:55
@36000
36000 merged commit e79e2fd into tractometry:main May 8, 2026
6 of 9 checks passed
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.

2 participants