Our FR5 dashboard shows a mode after the status read fails

OmarAbbott0056 · 28 Aug 2025, 03:21 UTC

Closed
OM
OmarAbbott0056
New SDK response shape, old wrapper. Failed read still shows a healthy-looking mode. Who should check the conversion?

12 replies

DI
DineshAli0212
Replying to OmarAbbott0056

Can the software maintainer compare the supported version's documented success and failure replies with the wrapper's assumptions?

11 points
OM
OmarAbbott0056
Replying to DineshAli0212

Maintainer already found it treats the returned container as a success flag.

12 points
DI
DineshAli0212
Replying to OmarAbbott0056

Is the displayed mode taken from that failed reply, or retained from an earlier successful one?

18 points
BE
BenAdams0109
Replying to OmarAbbott0056

Either way, show people that the read failed. Keeping the last mode can be useful, but not when it sits there pretending to be fresh news.

6 points
OM
OmarAbbott0056
Replying to DineshAli0212

Earlier successful value. Failed reply has no usable mode.

18 points
DI
DineshAli0212
Replying to OmarAbbott0056

Then test the good-to-failed transition as well as each response separately, so the display cannot retain current status after the failed read.

14 points
OM
OmarAbbott0056
Replying to DineshAli0212

Added to maintainer's test list. Current dashboard isn't being used to decide cell operation.

10 points
IM
ImranAdams0131
Replying to DineshAli0212

Our wrapper once guessed a mode from a missing field. Include malformed replies too, with no fallback to a plausible operational state.

11 points
OM
OmarAbbott0056
Replying to ImranAdams0131

Missing fields included. Unexpected shapes still need a case.

17 points
AA
AaronBrown0871
Replying to OmarAbbott0056

I would include startup before any valid reading, then a failed read after success and an unexpected response shape, checking that the operator can distinguish unavailable status from an actual reported mode in each case.

9 points
BE
BenAdams0109
Replying to AaronBrown0871

Who owns updating those examples when the SDK changes again? Otherwise the wrapper and the tests can grow old together while agreeing beautifully.

9 points
OM
OmarAbbott0056
Replying to BenAdams0109

Software maintainer owns them with the version record. Test version marks stale and unavailable explicitly. Controller comparison remains outstanding.

14 points

Discussion closed

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