Our status wrapper reads too much into a container (Universal Robots UR5e)

DavidCarter1041 · 19 Jul 2026, 14:08 UTC

Reply to discussion
DA
DavidCarter1041
Our dashboard for read-only mode reporting on Universal Robots UR5e expected a status value from the old SDK. The replacement environment returns a container with extra information, and our screen now shows a plausible status even when a request fails. This is in a bench integration setup, with an inspection coupon as a reference. I'm trying to fix how we interpret the answer.

11 replies

FE
FelixAli0197
Replying to DavidCarter1041

Can you compare sanitized success and failure responses against that SDK's documentation? The useful detail is which field your wrapper treats as status.

9 points
DA
DavidCarter1041
Replying to FelixAli0197

I've checked our saved examples. We're testing the outer container for a value, so the nonempty error response passes despite having no valid status.

20 points
FE
FelixAli0197
Replying to DavidCarter1041

Replay both offline with a version-specific parser. Failure should return unavailable. That pair should cover the adapter.

9 points
SA
SaraBaker0461
Replying to FelixAli0197

That covers this bug. What about a transport exception or missing field? Neither has to look like your saved error

-1 points
FE
FelixAli0197
Replying to SaraBaker0461

Fair point. Those two are a regression case, not full coverage. The adapter also needs documented error cases, transport exceptions and malformed responses exercised.

24 points
DA
DavidCarter1041
Replying to FelixAli0197

@FelixAli0197 Can our screen just say unavailable, but retain the reason?

7 points
FE
FelixAli0197
Replying to DavidCarter1041

@DavidCarter1041 Yes. One unavailable display state, distinct reasons in the result.

13 points
RE
RebeccaBrown0952
Replying to FelixAli0197

My setup kept showing its last successful value when the connection failed. It looked current to people; explicitly marking the current status unavailable made the difference.

18 points
DA
DavidCarter1041
Replying to RebeccaBrown0952

I've checked our display and it keeps the previous value without labelling it as an older observation. The parsing fix alone wouldn't address that.

20 points
DA
DavidCarter1041
Replying to DavidCarter1041

I've got an answer to the design question: parse the documented status for the identified SDK and keep failures unavailable. I'll call the display fix complete only when our offline examples show that behaviour on screen.

13 points
FE
FelixAli0197
Replying to DavidCarter1041

That design addresses the misleading answer directly, and your condition keeps the implementation claim tied to what the screen actually shows.

6 points

Add to the discussion

Welcome to Application Robot

Everyone can read the forum. Sign in or create an account to start a discussion, reply, or upload photos.

Forgot your password?

By creating an account, you agree to our Terms and Conditions and community guidelines. Read our Privacy Policy for how your information is handled.