acmerobotics / ftc-dashboard

React-based web dashboard designed for FTC
https://acmerobotics.github.io/ftc-dashboard
Other
168 stars 125 forks source link

[Bug] MultipleTelemetry.setAutoClear raises UnsupportedOperationException #126

Closed nash-pillai closed 1 year ago

nash-pillai commented 1 year ago

The following code makes me think that this is supposed to be a supported operation: image The opmode crashes when I call this method on MultipleTelemetry but not when I call it on the normal telemetry object.

rbrott commented 1 year ago

Thanks for the report. Indeed setAutoClear() is an unsupported operation on TelemetryAdapter. I intentionally don't support auto clear to allow multiple telemetry packets to be sent in the same "loop" and not have them clear each other, and the current code reflects that reality while (nominally) conforming to the Telemetry interface. Granted, I may have misunderstood the semantics here.