-
Notifications
You must be signed in to change notification settings - Fork 1.4k
DOC: Need to update instances of mayavi.core.api.Scene in docstrings #9919
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
Comments
yep, that's outdated. Care to open a pull request to fix it? I think the correct text is something like "pyvista Figure or mayavi Scene". I'm not sure whether |
I tried with notebook backend, but I get errors (below). Do you know what this error is about?
|
I think all 3D functions should work with the notebook backend, so I consider this is a bug, cc @GuillaumeFavelier |
Can you share the full code snippet @chapochn ? Did you set the 3d backend explicitly with (for example):
I tried my usual notebook (based on our examples) which uses
And I don't reproduce the bug: |
Hi GuillaumeFavelier, |
it's not supported on colab presently. I don't think anyone made it work
there so far
… |
Indeed support for Colab still requires development upstream ( You can follow the progress in: |
Hi, I installed the dev version of mne in a new env, and your code works there (but not in my usual env). My usual env where it is not working:
Working here, installed dev version of mne 0.24 with pip, and installed other packages it asked for.
I see my usual install doesn't have a separate line for pyvistaqt and doesn't mention ipyvtklink So I imagine the notebook 3D plotting was fixed in the dev version of mne? |
There have been many fixes since 0.23, probably some for the notebook, yes |
ok thanks! I can update the doc to mne.viz.plot_alignment with a PR |
Although I see it needs to be updated in many places in all 3d related functions |
Though @GuillaumeFavelier I think we have a bit of work to do in terms of saying what it is we actually return, IIRC in the PyVista cases it's some private-namespace object |
I see! I guess I will let you take care of it then, as it might be beyond my understanding at this point :) Thanks a lot! |
Fixed by #9945 |
Hello,
I am trying to better understand the visualization capabilities of mne.
The return of the function mne.viz.plot_alignment is said to be
fig : instance of mayavi.mlab.Figure
The mayavi figure.
On the other hand, I understand that mne is using one of 3 backends: mayavi, pyvista, notebook:
https://mne.tools/stable/generated/mne.viz.set_3d_backend.html
So should the documentation of mne.viz.plot_alignment be updated?
The text was updated successfully, but these errors were encountered: