Why does a failed UR5e read still show healthy?

RebeccaAdams0169 · 29 May 2025, 01:58 UTC

Closed
RE
RebeccaAdams0169
Rebuilt our dashboard environment. New response format, old wrapper. Failed requests now leave a convincing healthy indicator. Which checks should the developer put around the returned data?

13 replies

AN
AnilBarnes0549
Replying to RebeccaAdams0169

What does an actual failed response look like beside a successful one, using this installed version? Start there with its documentation. I'd want to see the error field and the data the wrapper is currently picking out.

19 points
RE
RebeccaAdams0169
Replying to AnilBarnes0549

Developer found it tests whether the whole response is non-empty. Even the error response passes.

16 points
AN
AnilBarnes0549
Replying to RebeccaAdams0169

That would explain it. Ask them to check documented success before reading the status payload, and keep the failed example for a regression test. An empty response needs a case too.

23 points
IS
IsaacBaker0520
Replying to RebeccaAdams0169

What happens to the last good value on the screen? Our stock display once kept an old number perfectly legible through a failed refresh, and everyone assumed it was current because nothing looked different.

8 points
RE
RebeccaAdams0169
Replying to IsaacBaker0520

Ours keeps it green. So that's two changes needed.

21 points
RE
RebeccaAdams0169
Replying to AnilBarnes0549

Draft now says current status unavailable. Last reading has its own age beside it. Developer is testing missing fields and unexpected shapes as well.

2 points
AN
AnilBarnes0549
Replying to RebeccaAdams0169

Can the display tell a valid negative status from a failed request? Those could both become false in a simple wrapper, but the operator needs different information for them.

15 points
CH
ChenBarnes0529
Replying to RebeccaAdams0169

Please try it from the other side of the bench. We had a screen whose stale-data warning was technically present, in tiny grey text under a huge green value. Lovely in the screenshot, useless where people stood.

2 points
RE
RebeccaAdams0169
Replying to ChenBarnes0529

Fair point. Warning is now beside the value, same readable size. Valid negative and failed request display differently.

13 points
AN
AnnaAbbott0068
Replying to RebeccaAdams0169

Keep the words too; changing green to another colour isn't much of an explanation.

11 points
AN
AnilBarnes0549
Replying to RebeccaAdams0169

Have you captured the version and response examples with the wrapper change? I don't want the next laptop rebuild to put you through this again because the working assumption lived only in the developer's head.

24 points
RE
RebeccaAdams0169
Replying to AnilBarnes0549

Yes. Offline cases pass. Waiting for the agreed read-only checks on the bench before release.

5 points
CH
ChenBarnes0529
Replying to RebeccaAdams0169

Good, let us know how the screen reads in use. I'm particularly interested in that old value after a failed refresh.

13 points

Discussion closed

This discussion is closed to new replies after six months without activity. Last activity: .