Skip to content
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

Pyvista foreground bug #7842

Closed
ualsbombe opened this issue May 28, 2020 · 0 comments · Fixed by #7843
Closed

Pyvista foreground bug #7842

ualsbombe opened this issue May 28, 2020 · 0 comments · Fixed by #7843

Comments

@ualsbombe
Copy link
Contributor

Describe the bug

〈Replace this text with a clear and concise description of the bug.〉

When changing the background to 'white' and foreground to 'black', text is not formatted accordingly (works as intended in mayavi) when using mne.viz.plot_source_estimates with pyvista

Steps to reproduce

import mne
from os.path import join

data_path = mne.datasets.sample.data_path()
subjects_dir = join(data_path, 'subjects')
filename = 'sample_audvis-meg'
stc = mne.read_source_estimate(join(data_path, 'MEG', 'sample', filename))

## works as intended

mne.viz.set_3d_backend('mayavi')
stc.plot('sample', subjects_dir=subjects_dir, background='white',
         foreground='black', figure=1)

## does not

mne.viz.set_3d_backend('pyvista')
stc.plot('sample', subjects_dir=subjects_dir, background='white',
         foreground='black', figure=2)

Expected results

That the numbers for the scale would be written in black, now they are (presumably) written in white

Actual results

Mayavi
image
Pyvista
image

Additional information

mne.sys_info()
Platform: Linux-5.3.0-53-generic-x86_64-with-debian-buster-sid
Python: 3.7.6 (default, Jan 8 2020, 19:59:22) [GCC 7.3.0]
Executable: /home/lau/miniconda3/envs/mnedev/bin/python
CPU: x86_64: 8 cores
Memory: 31.2 GB

mne: 0.21.dev0
numpy: 1.18.1 {blas=mkl_rt, lapack=mkl_rt}
scipy: 1.4.1
matplotlib: 3.1.3 {backend=Qt5Agg}

sklearn: 0.22.1
numba: 0.48.0
nibabel: 3.0.2
cupy: Not found
pandas: 1.0.2
dipy: 1.1.1
mayavi: 4.7.1
pyvista: 0.24.0
vtk: 8.2.0
PyQt5: 5.9.2

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

Successfully merging a pull request may close this issue.

2 participants