Skip to content

Conversation

@cbentejac
Copy link
Contributor

@cbentejac cbentejac commented Nov 10, 2023

Description

When an image cannot be loaded in the 2D Viewer, for example when visualizing the output of a node while an image whose viewpoint does not correspond to any output image is selected in the Image Gallery, the viewer remained in a loading state. The only way to get out of it was to try and load another image successfully.

This PR stops displaying a loading icon as soon as it has been detected that the image could not be loaded, basing itself on the FloatImageViewer's global status, which is updated in QtAliceVision: the viewer is cleared and the error message corresponding to the global status is displayed (in the case described above: "Missing File").

This relates to alicevision/QtAliceVision#57.

@cbentejac cbentejac self-assigned this Nov 10, 2023
@cbentejac cbentejac force-pushed the dev/catchMissingFile branch from 034cb11 to d1badc2 Compare November 13, 2023 11:48
@cbentejac cbentejac marked this pull request as ready for review November 13, 2023 11:50
@cbentejac cbentejac added this to the Meshroom 2024.1.0 milestone Nov 13, 2023
@cbentejac cbentejac marked this pull request as draft November 21, 2023 16:57
@cbentejac cbentejac force-pushed the dev/catchMissingFile branch from d1badc2 to d16b57c Compare December 13, 2023 11:11
@cbentejac cbentejac marked this pull request as ready for review December 13, 2023 11:22
@cbentejac cbentejac changed the title [Viewer] Display "Missing File" label when an image cannot be loaded [Viewer] Display error labels when an image cannot be loaded Dec 13, 2023
Based on QtAliceVision's global status for the FloatImageViewer, display
a specific label when an error has been encountered. The content of the
label depends on the type of the error.
@cbentejac cbentejac force-pushed the dev/catchMissingFile branch from d16b57c to 64c56e0 Compare December 14, 2023 10:56
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.

3 participants