Skip to content

Conversation

@cbentejac
Copy link
Contributor

@cbentejac cbentejac commented Nov 14, 2024

This PR upgrades all the plugins in QtAliceVision from Qt5.15.2 to Qt6.6.3.

It additionally integrates all the changes made in #72.

cbentejac and others added 20 commits November 13, 2024 15:55
In particular, the shaders for the point cloud display are updated to be
compatible with the Rhi backend.
…ility

The shaders for the FloatImageViewer are removed from C++ files and
compiled as a dedicated component. The ShaderImageViewer file is fully
removed. The code from the FloatImageViewer and the FloatTexture are
updated to reflect the changes from the Qt3D API.
When images in the PanoramaViewer are hovered, their gamma is increased
to highlight them and make it obvious to the user which image is currently
being seen.

This commit fixes an issue that occurred when the mouse was moving away
from the image it was hovering: the added gamma was never removed, meaning
that, by hovering one image enough times, said image would become totally
white. The highlighting effect now correctly increases and decreases the
hovered image's gamma level, including when the user clicks to edit the
panorama.
… shader

Clamp negative values in the FloatImageViewer fragment shader before
performing the successive `pow`. The result of `pow` is undefined when
it is provided negative values. On Linux platforms, it still gives the
expected result, but on Windows this leads to issues.
Source texture should always be initialized to avoid errors (texture binding) when no image is loaded, especially when `FloatImageViewer` is reactivated.
Allows to load and display image (albedo and normal) with a given light direction.
Shading is done using Blinn-Phong reflection model.
`std::powf` fails to compile with gcc which is not compliant with C++17
when it comes to libstdc++.

Additionally, static casts are performed to fix warnings.
@cbentejac cbentejac self-assigned this Nov 14, 2024
@cbentejac cbentejac mentioned this pull request Nov 14, 2024
@cbentejac cbentejac changed the title Qt6.6 upgrade Qt6.6.3 upgrade Nov 14, 2024
@fabiencastan fabiencastan merged commit 7c5aabd into develop Nov 22, 2024
@fabiencastan fabiencastan deleted the dev/qt6.6 branch November 22, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants