Skip to content
This repository has been archived by the owner on Mar 29, 2024. It is now read-only.

fix build with Qt 5.11 #41

Merged
merged 1 commit into from
Jun 8, 2018
Merged

fix build with Qt 5.11 #41

merged 1 commit into from
Jun 8, 2018

Conversation

rezso
Copy link
Contributor

@rezso rezso commented Jun 7, 2018

error: invalid use of incomplete type ‘class QHeaderView’
error: ‘QDoubleValidator’ was not declared in this scope
error: invalid use of incomplete type ‘class QButtonGroup’

@p-mary p-mary merged commit 2015dce into Tulip-Dev:master Jun 8, 2018
p-mary referenced this pull request in anlambert/talipot Jan 3, 2020
QOpenGL module is marked as deprecated since a while now so it is time
to remove its use from the Talipot codebase and promote the use of
QOpenGL* classes directly integrated in the QtGui module.

The big difference between QOpenGL and QtOpenGL from Qt5 is that all
rendering is performed in framebuffer objects, there is no more direct
rendering in the underlying os windows with its own OpenGL context.

Talipot OpenGL rendering also follows that idiom, all renderings are performed
offscreen using a shared OpenGL context. This also means that there is no
more QGLWidget as viewport for QGraphicsView. Talipot OpenGL scene are
now converted to QImage in order to display them using the default Qt raster
rendering engine. This should fixes the numerous rendering glitches observed
on MacOS.

First thing observed after the migration is a consequent performance boost
in OpenGL rendering when using an Intel GPU on a Linux host machine (especially
when selecting elements, it is now 10 times faster on debian stable).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants