-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
ENH: #10542 Adds __eq__() to DigMontage class and updates tests. #10942
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
ENH: #10542 Adds __eq__() to DigMontage class and updates tests. #10942
Conversation
|
Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴🏽♂️ |
…accordingly. Style Fixed
…accordingly. Style Fixed
agramfort
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@TzionaN can you add a what's new entry in latest.inc file? 🙏
|
Hey there! I can't seem to understand the failed build doc tests here, since there was only a minor adjustment to the doc, and it seems to match the requested syntax. |
DOC reorgonization/fix Co-authored-by: Richard Höchenberger <richard.hoechenberger@gmail.com>
Thanks for asking for help!
Looking at the CI runs for commit ab19cca, there were two things going on:
|
* upstream/main: FIX: ctf data can sometimes have no date time. (mne-tools#10957) replace check_version from _testing.py with the one in check.py (mne-tools#10958) ENH: mne-tools#10542 Adds __eq__() to DigMontage class and updates tests. (mne-tools#10942) Use HiDPI icons with PyQt5/PySide2 (mne-tools#10956)
Reference issue
Fixes #10542
Does: