-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[BUG, MRG] Fix tutorial + example show_view to use correct keyword args #9599
Conversation
Makes sense, to be safe can you make sure you touch/change all six failed examples in https://app.circleci.com/pipelines/github/mne-tools/mne-python/9192/workflows/843a1e8e-b046-48bc-9909-908f9b29b317/jobs/31440 EDIT: so that CircleCI runs and we make sure the errors are gone |
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, will merge once CircleCI comes back happy. Thanks for the quick fix @alexrockhill !
@@ -280,8 +279,7 @@ def plot_approach(power, n): | |||
brain.add_foci(vertices[0][0], coords_as_verts=True, hemi='lh', color='b') | |||
brain.add_foci(vertices[1][0], coords_as_verts=True, hemi='rh', color='b') | |||
# Rotate the view and add a title. | |||
brain.show_view(view={'azimuth': 0, 'elevation': 0, 'distance': 550, | |||
'focalpoint': [0, 0, 0]}) | |||
brain.show_view(azimuth=0, elevation=0, distance=550, focalpoint=(0, 0, 0)) |
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.
What a nicer API :)
@larsoner, do you know what's going on with this failed test? |
Looks unrelated to your changes, I'm guessing it's VTK 9.0.3's fault. I'll open a PR to try to fix it |
Thanks @alexrockhill ! |
…gs (mne-tools#9599) * fix tutorial + example show_view * clean up paths to touch examples * seg fault
This is in reference to #9598.
Also if there were two hemispheres but the same view, it would do the depreciation warning so I fixed that.