Skip to content

Add multi-origin SSS method to Maxwell Filtering#13417

Draft
xannnimal wants to merge 11 commits intomne-tools:mainfrom
xannnimal:implement-mSSS
Draft

Add multi-origin SSS method to Maxwell Filtering#13417
xannnimal wants to merge 11 commits intomne-tools:mainfrom
xannnimal:implement-mSSS

Conversation

@xannnimal
Copy link
Copy Markdown

@xannnimal xannnimal commented Sep 11, 2025

Reference issue #13339 Add multi-origin SSS method to Maxwell Filtering

Begin implementation of mSSS basis calculation.
Added function preprocessing/fit_spheres_to_mri.py to calculate optimized spheres based on MRI, returns centers of spheres
Added preliminary function in preprocessing/maxwell.py called combine_sss_basis which takes in two internal SSS matrices and combines into one internal basis.

What does this implement?

Adds functionality for using SSS with on-scalp MEG sensor systems (eg OPM)

Additional information

This is a draft and need to add functionality to accept multiple origins in maxwell.py and add functionality to calculate multiple SSS matrices using these origins, entered from user
obtained from preprocessing/fit_spheres_to_mri.py or otherwise. These multiple SSS matrices can then be used in combine_sss_basis. @larsoner any suggestions/comments for implementing this part would be awesome! Thanks!

EDIT by @larsoner: Ref to https://iopscience.iop.org/article/10.1088/1361-6560/ade6ba

@welcome
Copy link
Copy Markdown

welcome Bot commented Sep 11, 2025

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴

@drammock
Copy link
Copy Markdown
Member

@xannnimal this PR has conflicts with main, and also has a bunch of unrelated changes showing up in the diff. LMK if you want help rebasing and we can schedule a short pair-programming session to tackle it together.

@xannnimal
Copy link
Copy Markdown
Author

@drammock Thank you Dan, that would be awesome!! I have checked that the functionality of the mSSS calculation and origins are working as expected now

@drammock
Copy link
Copy Markdown
Member

@drammock Thank you Dan, that would be awesome!! I have checked that the functionality of the mSSS calculation and origins are working as expected now

I'll ping you via email to schedule

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