Client: improve error reporting #56372

New Issue

Sybren A. Stüvel · 2018-08-14T09:39:15+02:00

Sybren A. Stüvel commented

2018-08-14 09:39:15 +02:00

Benchmark submission errors are now reported with the Operator.report() function. This has a few downsides:

The error is shown in a popup, which disappears after the user moved the mouse too much. Since submission of the benchmark can happen in the background (while the user is still looking at the "you can close me" page in their webbrowser), errors can go unnoticed.
Long text isn't wrapped, but rather causes the title of the popup to be invisible (due to being positioned on coordinates outside the window and thus not drawn).

As a possible solution we could avoid the report() function and write the error message to a permanently-shown and word-wrapping UI element. The submission button should also change text, like "Retry Submission" or something along those lines.

Benchmark submission errors are now reported with the `Operator.report()` function. This has a few downsides: - The error is shown in a popup, which disappears after the user moved the mouse too much. Since submission of the benchmark can happen in the background (while the user is still looking at the "you can close me" page in their webbrowser), errors can go unnoticed. - Long text isn't wrapped, but rather causes the title of the popup to be invisible (due to being positioned on coordinates outside the window and thus not drawn). As a possible solution we could avoid the `report()` function and write the error message to a permanently-shown and word-wrapping UI element. The submission button should also change text, like "Retry Submission" or something along those lines.

Sybren A. Stüvel commented

2018-08-14 09:39:15 +02:00

Added subscriber: @dr.sybren

Sybren A. Stüvel self-assigned this 2018-08-14 15:05:17 +02:00

blender-admin commented

2018-08-14 15:05:37 +02:00

This issue was referenced by infrastructure/blender-benchmark-bundle@7b88d7704c

This issue was referenced by infrastructure/blender-benchmark-bundle@7b88d7704c222107c6468d88b7eed5a14152796d

Sybren A. Stüvel commented

2018-08-14 15:09:19 +02:00

Changed status from 'Open' to: 'Resolved'

Sybren A. Stüvel closed this issue