Meshroom icon indicating copy to clipboard operation
Meshroom copied to clipboard

Published 20 hours ago • verified publisher icon alicevision

[ui] Reduce confusion when qml loading fails

Open p12tic opened this issue 1 year ago • 1 comments

Currently we disable all logging out of qml by default. This is problematic in case qml loading fails for any reason (e.g. missing dependencies) as the user will be presented with a message that is not actionable: "QQmlApplicationEngine failed to load component". There is no way to understand what's happening unless the user knows about MESHROOM_OUTPUT_QML_WARNINGS.

This is improved by checking for presence of "QQmlApplicationEngine failed to load component" and warning if qml logging is disabled in that case.

There has been many issues with this log message and no further logs in the bug tracker, so I'm not the only one who hit this issue. Hopefully this improves debugging experience for everyone.

p12tic avatar Jul 14 '22 10:07 p12tic

@fabiencastan Just a friendly ping :-)

p12tic avatar Jul 22 '22 01:07 p12tic

@fabiencastan friendly ping :-) I think this PR is good to go in.

p12tic avatar Aug 25 '22 12:08 p12tic