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

Make it pass desktop-file-validate #12

Closed
wants to merge 1 commit into from

Conversation

probonopd
Copy link

@probonopd probonopd commented Aug 19, 2017

Reference: https://travis-ci.org/AppImage/AppImageHub/builds/266251154#L544

Version= is not for the version of the application, according to the XDG desktop spec.

@anlambert anlambert changed the base branch from master to tulip-next-playground August 21, 2017 18:38
anlambert added a commit that referenced this pull request Sep 9, 2017
@anlambert
Copy link
Contributor

fix in 607bee6, thanks !

@anlambert anlambert closed this Sep 9, 2017
bpinaud referenced this pull request Jan 29, 2019
TravisCI: Migrate Linux builds from trusty to xenial
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