Skip to content

Adding MEM localization method to MNE-tools #12659

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 14, 2024

Conversation

ilianAZZ
Copy link
Contributor

This is a method to localize the superficial generators with their spatial extent using the Maximum Entropy on the Mean.

References :

@ilianAZZ ilianAZZ requested a review from hoechenberger as a code owner June 13, 2024 02:41
Copy link

welcome bot commented Jun 13, 2024

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

Copy link
Member

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

Link looks good!

https://output.circle-artifacts.com/output/job/871ae2f6-c9c0-4710-806e-1926a1cd9708/artifacts/0/html/install/mne_tools_suite.html

Can you add your name to docs/changes/names.inc and add a tiny doc/changes/12659.other.rst mentioning this doc update? Then we can merge

@ilianAZZ
Copy link
Contributor Author

Hi @larsoner

Thanks for your feedback, it's done !

@ilianAZZ
Copy link
Contributor Author

Done!

@ilianAZZ ilianAZZ requested review from larsoner and drammock June 14, 2024 00:17
@drammock drammock merged commit d0eff79 into mne-tools:main Jun 14, 2024
28 checks passed
Copy link

welcome bot commented Jun 14, 2024

🎉 Congrats on merging your first pull request! 🥳 Looking forward to seeing more from you in the future! 💪

@ilianAZZ ilianAZZ deleted the add-MEM-tool branch June 14, 2024 17:23
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