bluerobotics / cockpit

An intuitive and customizable cross-platform ground control station for remote vehicles of all types.
https://blueos.cloud/docs/extensions/cockpit/1.0/overview
Other
54 stars 20 forks source link

Eventing #594

Open shgavman opened 10 months ago

shgavman commented 10 months ago

Hi Rafael,

I suspect a lot of us require an eventing tool in our navigation software - this is needed when carrying out inspection. A button can be assigned to logging an event. Pressing that button should capture all available telemetry, a screenshot (either the full screen including other devices such as sonar or only the camera window(s), and operator comment. It should be possible to export the data and ideally export a report.

Suggested tables and fields:

Project Table

Dive Log Table

Anomaly Log Table

Perhaps a test table to for Cathodic Prection Probes or Ultrasonic thickness gauge for example. That could come later. I think the above would be a good minimum viable product.

rafaellehmkuhl commented 10 months ago

Hey Gav! Thanks for opening the issue. It's indeed a feature that would probably benefit a lot of people and that we do want to have.

Do you think, on the event triggering (e.g.: joystick button click) we should log only one point or have several points during a period of time (e.g.: when the button is clicked we bring to the report a set of 10 telemetry points around this timestamp, including points before the button click, so the user can have data about events that just happened)?

ES-Alexander commented 10 months ago

This idea is definitely useful, but I think it likely works better as an add-on rather than part of the core Cockpit functionality, since

I think it's important to set a precedent of minimising complexity and bloat in Cockpit for users who don't need it - the core should be a lightweight but powerful general platform, which users and developers can then build upon and extend to suit their more specific applications.

That being said, the functionality does seem sufficiently useful that it's not unreasonable for BR to put some resources into developing such an add-on - I'm just commenting on where I think it belongs.

ES-Alexander commented 10 months ago

609 would be relevant to this, I think, although we may also need to introduce the concept of a hide-able widget (since we'd want some interface to add notes to when events are triggered, but it should then go away during normal operation).