Cannot render vispy when switching from non-vispy widget
Sorry for the unclear issue title.
Note that this issue is only with PyQt6 or PySide6, and not present in PyQt5 or PySide2.
Here's the output of vispy.sys_info()
WARNING: pyqt5 already imported, cannot switch to pyqt6
WARNING: pyqt5 already imported, cannot switch to pyside2
WARNING: pyqt5 already imported, cannot switch to pyside6
Platform: Windows-10-10.0.19044-SP0
Python: 3.9.13 (tags/v3.9.13:6de2ca5, May 17 2022, 16:36:42) [MSC v.1929 64 bit (AMD64)]
NumPy: 1.23.3
Backend: PyQt5
pyqt4: None
pyqt5: ('PyQt5', '5.15.7', '5.15.2')
pyqt6: ('PyQt6', '6.3.1', '6.3.1')
pyside: None
pyside2: ('PySide2', '5.15.2.1', '5.15.2')
pyside6: ('PySide6', '6.3.1', '6.3.1')
pyglet: None
glfw: None
sdl2: None
wx: None
egl: None
osmesa: None
tkinter: None
jupyter_rfb: None
_test: None
GL version: '4.6.14800 Compatibility Profile Context 22.5.1 30.0.15021.11005'
MAX_TEXTURE_SIZE: 16384
Extensions: 'GL_AMDX_debug_output GL_AMD_blend_minmax_factor GL_AMD_conservative_depth GL_AMD_debug_output GL_AMD_depth_clamp_separate GL_AMD_draw_buffers_blend GL_AMD_framebuffer_sample_positions GL_AMD_gcn_shader GL_AMD_gpu_shader_half_float GL_AMD
_gpu_shader_int16 GL_AMD_gpu_shader_int64 GL_AMD_interleaved_elements GL_AMD_multi_draw_indirect GL_AMD_name_gen_delete GL_AMD_occlusion_query_event GL_AMD_performance_monitor GL_AMD_pinned_memory GL_AMD_query_buffer_object GL_AMD_sample_positions GL
_AMD_seamless_cubemap_per_texture GL_AMD_shader_atomic_counter_ops GL_AMD_shader_stencil_export GL_AMD_shader_stencil_value_export GL_AMD_shader_trace GL_AMD_shader_trinary_minmax GL_AMD_sparse_texture GL_AMD_sparse_texture_pool GL_AMD_stencil_operat
ion_extended GL_AMD_texture_cube_map_array GL_AMD_texture_texture4 GL_AMD_transform_feedback3_lines_triangles GL_AMD_transform_feedback4 GL_AMD_vertex_shader_layer GL_AMD_vertex_shader_viewport_index GL_ARB_ES2_compatibility GL_ARB_ES3_1_compatibilit
y GL_ARB_ES3_compatibility GL_ARB_arrays_of_arrays GL_ARB_base_instance GL_ARB_bindless_texture GL_ARB_blend_func_extended GL_ARB_buffer_storage GL_ARB_clear_buffer_object GL_ARB_clear_texture GL_ARB_clip_control GL_ARB_color_buffer_float GL_ARB_comp
atibility GL_ARB_compressed_texture_pixel_storage GL_ARB_compute_shader GL_ARB_conditional_render_inverted GL_ARB_conservative_depth GL_ARB_copy_buffer GL_ARB_copy_image GL_ARB_cull_distance GL_ARB_debug_output GL_ARB_depth_buffer_float GL_ARB_depth_
clamp GL_ARB_depth_texture GL_ARB_derivative_control GL_ARB_direct_state_access GL_ARB_draw_buffers GL_ARB_draw_buffers_blend GL_ARB_draw_elements_base_vertex GL_ARB_draw_indirect GL_ARB_draw_instanced GL_ARB_enhanced_layouts GL_ARB_explicit_attrib_l
ocation GL_ARB_explicit_uniform_location GL_ARB_fragment_coord_conventions GL_ARB_fragment_layer_viewport GL_ARB_fragment_program GL_ARB_fragment_program_shadow GL_ARB_fragment_shader GL_ARB_framebuffer_no_attachments GL_ARB_framebuffer_object GL_ARB
_framebuffer_sRGB GL_ARB_geometry_shader4 GL_ARB_get_program_binary GL_ARB_get_texture_sub_image GL_ARB_gl_spirv GL_ARB_gpu_shader5 GL_ARB_gpu_shader_fp64 GL_ARB_half_float_pixel GL_ARB_half_float_vertex GL_ARB_imaging GL_ARB_indirect_parameters GL_A
RB_instanced_arrays GL_ARB_internalformat_query GL_ARB_internalformat_query2 GL_ARB_invalidate_subdata GL_ARB_map_buffer_alignment GL_ARB_map_buffer_range GL_ARB_multi_bind GL_ARB_multi_draw_indirect GL_ARB_multisample GL_ARB_multitexture GL_ARB_occl
usion_query GL_ARB_occlusion_query2 GL_ARB_parallel_shader_compile GL_ARB_pipeline_statistics_query GL_ARB_pixel_buffer_object GL_ARB_point_parameters GL_ARB_point_sprite GL_ARB_polygon_offset_clamp GL_ARB_program_interface_query GL_ARB_provoking_ver
tex GL_ARB_query_buffer_object GL_ARB_robust_buffer_access_behavior GL_ARB_sample_shading GL_ARB_sampler_objects GL_ARB_seamless_cube_map GL_ARB_seamless_cubemap_per_texture GL_ARB_separate_shader_objects GL_ARB_shader_atomic_counter_ops GL_ARB_shade
r_atomic_counters GL_ARB_shader_ballot GL_ARB_shader_bit_encoding GL_ARB_shader_draw_parameters GL_ARB_shader_group_vote GL_ARB_shader_image_load_store GL_ARB_shader_image_size GL_ARB_shader_objects GL_ARB_shader_precision GL_ARB_shader_stencil_expor
t GL_ARB_shader_storage_buffer_object GL_ARB_shader_subroutine GL_ARB_shader_texture_image_samples GL_ARB_shader_texture_lod GL_ARB_shader_viewport_layer_array GL_ARB_shading_language_100 GL_ARB_shading_language_420pack GL_ARB_shading_language_includ
e GL_ARB_shading_language_packing GL_ARB_shadow GL_ARB_shadow_ambient GL_ARB_sparse_buffer GL_ARB_sparse_texture GL_ARB_spirv_extensions GL_ARB_stencil_texturing GL_ARB_sync GL_ARB_tessellation_shader GL_ARB_texture_barrier GL_ARB_texture_border_clam
p GL_ARB_texture_buffer_object GL_ARB_texture_buffer_object_rgb32 GL_ARB_texture_buffer_range GL_ARB_texture_compression GL_ARB_texture_compression_bptc GL_ARB_texture_compression_rgtc GL_ARB_texture_cube_map GL_ARB_texture_cube_map_array GL_ARB_text
ure_env_add GL_ARB_texture_env_combine GL_ARB_texture_env_crossbar GL_ARB_texture_env_dot3 GL_ARB_texture_float GL_ARB_texture_gather GL_ARB_texture_mirror_clamp_to_edge GL_ARB_texture_mirrored_repeat GL_ARB_texture_multisample GL_ARB_texture_non_pow
er_of_two GL_ARB_texture_query_levels GL_ARB_texture_query_lod GL_ARB_texture_rectangle GL_ARB_texture_rg GL_ARB_texture_rgb10_a2ui GL_ARB_texture_snorm GL_ARB_texture_stencil8 GL_ARB_texture_storage GL_ARB_texture_storage_multisample GL_ARB_texture_
swizzle GL_ARB_texture_view GL_ARB_timer_query GL_ARB_transform_feedback2 GL_ARB_transform_feedback3 GL_ARB_transform_feedback_instanced GL_ARB_transform_feedback_overflow_query GL_ARB_transpose_matrix GL_ARB_uniform_buffer_object GL_ARB_vertex_array
_bgra GL_ARB_vertex_array_object GL_ARB_vertex_attrib_64bit GL_ARB_vertex_attrib_binding GL_ARB_vertex_buffer_object GL_ARB_vertex_program GL_ARB_vertex_shader GL_ARB_vertex_type_10f_11f_11f_rev GL_ARB_vertex_type_2_10_10_10_rev GL_ARB_viewport_array
GL_ARB_window_pos GL_ATI_draw_buffers GL_ATI_envmap_bumpmap GL_ATI_fragment_shader GL_ATI_separate_stencil GL_ATI_texture_compression_3dc GL_ATI_texture_env_combine3 GL_ATI_texture_float GL_ATI_texture_mirror_once GL_EXT_abgr GL_EXT_bgra GL_EXT_bind
able_uniform GL_EXT_blend_color GL_EXT_blend_equation_separate GL_EXT_blend_func_separate GL_EXT_blend_minmax GL_EXT_blend_subtract GL_EXT_compiled_vertex_array GL_EXT_copy_buffer GL_EXT_copy_texture GL_EXT_depth_bounds_test GL_EXT_direct_state_acces
s GL_EXT_draw_buffers2 GL_EXT_draw_instanced GL_EXT_draw_range_elements GL_EXT_fog_coord GL_EXT_framebuffer_blit GL_EXT_framebuffer_multisample GL_EXT_framebuffer_object GL_EXT_framebuffer_sRGB GL_EXT_geometry_shader4 GL_EXT_gpu_program_parameters GL
_EXT_gpu_shader4 GL_EXT_histogram GL_EXT_memory_object GL_EXT_memory_object_win32 GL_EXT_multi_draw_arrays GL_EXT_packed_depth_stencil GL_EXT_packed_float GL_EXT_packed_pixels GL_EXT_pixel_buffer_object GL_EXT_point_parameters GL_EXT_polygon_offset_c
lamp GL_EXT_provoking_vertex GL_EXT_rescale_normal GL_EXT_secondary_color GL_EXT_semaphore GL_EXT_semaphore_win32 GL_EXT_separate_specular_color GL_EXT_shader_image_load_store GL_EXT_shader_integer_mix GL_EXT_shadow_funcs GL_EXT_stencil_wrap GL_EXT_s
ubtexture GL_EXT_texgen_reflection GL_EXT_texture3D GL_EXT_texture_array GL_EXT_texture_buffer_object GL_EXT_texture_compression_bptc GL_EXT_texture_compression_latc GL_EXT_texture_compression_rgtc GL_EXT_texture_compression_s3tc GL_EXT_texture_cube_
map GL_EXT_texture_edge_clamp GL_EXT_texture_env_add GL_EXT_texture_env_combine GL_EXT_texture_env_dot3 GL_EXT_texture_filter_anisotropic GL_EXT_texture_integer GL_EXT_texture_lod GL_EXT_texture_lod_bias GL_EXT_texture_mirror_clamp GL_EXT_texture_obj
ect GL_EXT_texture_rectangle GL_EXT_texture_sRGB GL_EXT_texture_sRGB_R8 GL_EXT_texture_sRGB_RG8 GL_EXT_texture_sRGB_decode GL_EXT_texture_shared_exponent GL_EXT_texture_snorm GL_EXT_texture_storage GL_EXT_texture_swizzle GL_EXT_timer_query GL_EXT_tra
nsform_feedback GL_EXT_vertex_array GL_EXT_vertex_array_bgra GL_EXT_vertex_attrib_64bit GL_IBM_texture_mirrored_repeat GL_KHR_context_flush_control GL_KHR_debug GL_KHR_no_error GL_KHR_parallel_shader_compile GL_KHR_robust_buffer_access_behavior GL_KH
R_robustness GL_KTX_buffer_region GL_NV_alpha_to_coverage_dither_control GL_NV_blend_square GL_NV_conditional_render GL_NV_copy_depth_to_color GL_NV_copy_image GL_NV_depth_buffer_float GL_NV_explicit_multisample GL_NV_float_buffer GL_NV_half_float GL
_NV_primitive_restart GL_NV_shader_atomic_int64 GL_NV_texgen_reflection GL_NV_texture_barrier GL_OES_EGL_image GL_SGIS_generate_mipmap GL_SGIS_texture_edge_clamp GL_SGIS_texture_lod GL_SUN_multi_draw_arrays GL_WIN_swap_hint WGL_EXT_swap_control'
And here's a minimal viable example of what I'm trying to get to work.
from PyQt6.QtCore import QTimer, pyqtSignal
from PyQt6.QtWidgets import QApplication, QMainWindow, QVBoxLayout, QWidget, QPushButton
from vispy import scene
import numpy as np
import sys
N = 200
pos = np.zeros((N, 2), dtype=np.float32)
x_lim = [50., 750.]
y_lim = [-2., 2.]
pos[:, 0] = np.linspace(x_lim[0], x_lim[1], N)
pos[:, 1] = np.random.normal(size=N)
class ChildButtonVispyWidget(QWidget):
def __init__(self, set_vispy_widget_signal):
super().__init__()
button = QPushButton("Start Vispy!")
button.clicked.connect(set_vispy_widget_signal)
self.canvas = scene.SceneCanvas()
self.line = scene.Line(pos=pos)
self.line.parent = self.canvas.central_widget
layout = QVBoxLayout()
layout.addWidget(button)
layout.addWidget(self.canvas.native)
self.setLayout(layout)
self.timer = QTimer()
self.timer.setInterval(1000)
self.timer.timeout.connect(self.update)
self.timer.start()
def update(self):
global pos
pos[:, 1] = np.random.normal(size=N, scale=500)
self.line.set_data(pos)
class ChildVispyWidget(QWidget):
def __init__(self):
super().__init__()
self.canvas = scene.SceneCanvas()
self.line = scene.Line(pos=pos)
self.line.parent = self.canvas.central_widget
layout = QVBoxLayout()
layout.addWidget(self.canvas.native)
self.setLayout(layout)
self.timer = QTimer()
self.timer.setInterval(1000)
self.timer.timeout.connect(self.update)
self.timer.start()
def update(self):
global pos
pos[:, 1] = np.random.normal(size=N, scale=500)
self.line.set_data(pos)
class ChildButtonWidget(QWidget):
def __init__(self, set_vispy_widget_signal):
super(ChildButtonWidget, self).__init__()
button = QPushButton("Start Vispy!")
button.clicked.connect(set_vispy_widget_signal)
layout = QVBoxLayout()
layout.addWidget(button)
self.setLayout(layout)
class MainWindow(QMainWindow):
start_vispy = pyqtSignal()
def __init__(self):
super().__init__()
self.vispy_widget = None
print("MainWindow started")
self.setWindowTitle("Main window for vispy test")
self.start_vispy.connect(self.set_child_vispy_widget)
self.set_child_btn_widget()
#self.set_child_vispy_widget()
#self.set_child_vispy_btn_widget()
def set_child_btn_widget(self):
print("Setting child widget")
btn_widget = ChildButtonWidget(self.start_vispy)
self.setCentralWidget(btn_widget)
def set_child_vispy_widget(self):
print("Setting vispy widget")
vispy_widget = ChildVispyWidget()
self.setCentralWidget(vispy_widget)
def set_child_vispy_btn_widget(self):
print("Setting vispy + button widget")
vispy_btn_widget = ChildButtonVispyWidget(self.start_vispy)
self.setCentralWidget(vispy_btn_widget)
app = QApplication(sys.argv)
window = MainWindow()
window.show()
app.exec()
So, here's the issue:
When the central widget is switched from a widget which doesn't have a vispy widget in its layout, switching to a vispy widget fails with WARNING: void __cdecl QPlatformBackingStore::composeAndFlush(class QWindow *,const class QRegion &,const class QPoint &,class QPlatformTextureList *,bool) no opengl support set
However, this is not an issue when the child I'm trying switch to is set as the default child on PyQt/PySide startup, nor is it an issue when I switch from a child widget which already has a vispy child.
You can comment out one of self.set_child_vispy_widget() or self.set_child_vispy_btn_widget() at end of MainWindow's init to see what I'm talking about.
This issue persists with PySide6 as well, but as noted at the beginning, cannot replicate it on PyQt5 or PySide6. Be sure to replace pyqtSignal() with Signal() when you switch to PySide6 or PySide2, and app.exec() to app.exec_() when you switch to older versions.
So my gut reaction is that this is a bug in Qt6 and is something we (vispy maintainers) won't be able to fix. The code in the vispy backend for all supported Qt libraries is pretty similar and doesn't differ much between Qt5 and Qt6. From what I remember most of the differences between Qt5 and 6 are the name changes for things like keyboard modifier enums. That said...
You may have tried this or I missed it in your description, but what if you have a QWidget as your central widget and then the vispy and button widgets you're using now are just children of that widget. Does that change the results at all? I've never changed the central widget in a Qt application after initially setting it. Obviously you're able to do it sometimes so it isn't explicitly disallowed, but I'm curious if it might be an easy workaround.
Another thing, you pass the start_vispy signal to some of the sub-widgets. This seems error-prone or at the very least like a design that could be improved. I would think it would be better to have a signal on the individual sub-widgets that you connect to the necessary methods of the main window. Eh, it is probably fine, it just makes me feel weird.
Lastly, you pass start_vispy and then connect the button clicked signal to it. I didn't know you could connect a signal to another signal like that. Cool!
I might not be that well versed in Qt, but how would one accomplish what you're asking by this?
what if you have a QWidget as your central widget and then the vispy and button widgets you're using now are just children of that widget.
I feel like that's what is done when you comment out self.set_child_vispy_btn_widget(). I don't know of anyway of changing layouts (to accomplish 'moving to a new view' effect) the without changing the central widget to something that has a different layout. Is there a way of destroying current layout and adding a new layout?
I would think it would be better to have a signal on the individual sub-widgets that you connect to the necessary methods of the main window.
You mean using something like changing the line button.clicked.connect(self.set_child_vispy_widget) to button.clicked.connect(self.parent().set_child_vispy_widget ?
There seems to be a paradigm of writing Qt apps in Python that I'm missing. I would appreciate any pointers to different ways of laying out complex Qt screens in Python. Thanks in advance!
Lastly, you pass start_vispy and then connect the button clicked signal to it. I didn't know you could connect a signal to another signal like that. Cool!
And this is the hacky way I do this. If I use a signal more than once or have to share signals between widgets that are siblings so can't pass signals between each other, I add a signal to sub-classed QApplication as described here: https://stackoverflow.com/a/73073783
I think I'll answer some of this out of order...
There seems to be a paradigm of writing Qt apps in Python that I'm missing. I would appreciate any pointers to different ways of laying out complex Qt screens in Python. Thanks in advance!
I'm definitely not an expert, but I know when code has a "smell" that just doesn't seem right.
And this is the hacky way I do this. If I use a signal more than once or have to share signals between widgets that are siblings so can't pass signals between each other, I add a signal to sub-classed QApplication as described here: https://stackoverflow.com/a/73073783
Speaking of code smell, this idea seems like there should be a better way. But again, not an expert, and there are definitely types of Qt applications I have not made that may make more sense with this QApplication idea, but it seems odd.
I feel like that's what is done when you comment out self.set_child_vispy_btn_widget(). I don't know of anyway of changing layouts (to accomplish 'moving to a new view' effect) the without changing the central widget to something that has a different layout. Is there a way of destroying current layout and adding a new layout?
Maybe you're right, ChildButtonVispyWidget is doing that and to adapt it to what I was saying would be that you change the child-widgets of that ChildButtonVispyWidget instance rather than changing the central widget of the main window. I think you can "just" set a new layout similar to how you did when you originally created the widget. I've never done this myself as when I needed to change widgets in the past I would use something like a stacked or tabbed widget and switch between the widgets I wanted to look at. This StackOverflow answer discusses how you could easily set an old layout onto a temporary QWidget and have it destroyed:
https://stackoverflow.com/a/10439207/433202
The main idea though is:
self.central_widget = QtGui.QWidget()
self.central_widget.setLayout(self.some_layout)
self.setCentralWidget(self.central_widget)
and then later calling self.central_widget.setLayout again with a new layout.
Note that this whole idea of using an "intermediate" QWidget was just a guess and something I was hoping could be tested to see if it helps with the issue. But also, as you said, maybe you're already doing this with the ChildButtonVispyWidget and just need to adjust it a little.