Skip to content

MRG, FIX: retain reference to lasso.selection without blocking - #8469

Merged
larsoner merged 1 commit into
mne-tools:masterfrom
drammock:fix-lasso-selection
Nov 5, 2020
Merged

MRG, FIX: retain reference to lasso.selection without blocking#8469
larsoner merged 1 commit into
mne-tools:masterfrom
drammock:fix-lasso-selection

Conversation

@drammock

@drammock drammock commented Nov 2, 2020

Copy link
Copy Markdown
Member

This makes it so that the list of sensors returned by plot_sensors(..., kind='select') will continue to be updated by figure interactions even if block=False. It restores previous behavior that was broken by #7955.

@drammock

drammock commented Nov 2, 2020

Copy link
Copy Markdown
Member Author

@mmagnuski this works for me in local testing, can you confirm?

@drammock
drammock force-pushed the fix-lasso-selection branch from 1908c2e to 169f774 Compare November 3, 2020 20:03
@mmagnuski

Copy link
Copy Markdown
Member

@drammock I can confirm this fixes the problem, thanks!

@drammock drammock changed the title FIX: retain reference to lasso.selection without blocking MRG, FIX: retain reference to lasso.selection without blocking Nov 3, 2020
@drammock

drammock commented Nov 3, 2020

Copy link
Copy Markdown
Member Author

the Style failure has already been fixed in b2b63fd from #8477
ready for review/merge

@larsoner

larsoner commented Nov 3, 2020

Copy link
Copy Markdown
Member

Travis minimal fails, there is a fix in #8477

@larsoner

larsoner commented Nov 3, 2020

Copy link
Copy Markdown
Member

(but it will conflict with the changes here)

@drammock

drammock commented Nov 3, 2020

Copy link
Copy Markdown
Member Author

(but it will conflict with the changes here)

we can merge that one first, I don't mind rebasing.

@drammock
drammock force-pushed the fix-lasso-selection branch from 169f774 to 950caed Compare November 4, 2020 15:23
Comment thread mne/viz/tests/test_raw.py
@drammock
drammock force-pushed the fix-lasso-selection branch from 950caed to ffa685b Compare November 4, 2020 22:10
@drammock
drammock force-pushed the fix-lasso-selection branch from ffa685b to 804dc47 Compare November 4, 2020 22:33
@larsoner
larsoner merged commit 7a87d0c into mne-tools:master Nov 5, 2020
@drammock
drammock deleted the fix-lasso-selection branch November 5, 2020 16:39
@larsoner

larsoner commented Nov 5, 2020

Copy link
Copy Markdown
Member

Thanks @drammock !

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