Skip to content

FIX: Expose _set_3d_view and _set_3d_title#8392

Merged
larsoner merged 6 commits intomne-tools:masterfrom
GuillaumeFavelier:notebook_private_func
Oct 20, 2020
Merged

FIX: Expose _set_3d_view and _set_3d_title#8392
larsoner merged 6 commits intomne-tools:masterfrom
GuillaumeFavelier:notebook_private_func

Conversation

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor

This PR exposes the functions _set_3d_view() and _set_3d_title() to the notebook 3d backend.

Related to #8388

@GuillaumeFavelier GuillaumeFavelier self-assigned this Oct 20, 2020
@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

Let me know if this solves your issue @timonmerk

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

This is ready for reviews @agramfort, @larsoner

@larsoner
Copy link
Copy Markdown
Member

Can you modify the notebook test such that it has better coverage?

@larsoner
Copy link
Copy Markdown
Member

@GuillaumeFavelier do we need to backport this?

@larsoner larsoner merged commit 8fa5aa5 into mne-tools:master Oct 20, 2020
@GuillaumeFavelier GuillaumeFavelier deleted the notebook_private_func branch October 20, 2020 13:57
larsoner pushed a commit that referenced this pull request Oct 20, 2020
* Expose _set_3d_view and _set_3d_title

* Fix style

* Improve coverage

* Improve coverage

* Improve coverage

* Improve coverage
@larsoner
Copy link
Copy Markdown
Member

... it seemed to work without issue so I backported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants