bluerobotics / bluerobotics/BlueOS
core: frontend: vehicle-setup: reflect output pin function
- Dominant language
- Vue
- Stars
- 453
- Forks
- 151
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 174
Description
### Current behaviour
#3122 introduced some additional configuration options, but does so assuming every output pin is configured as some form of standard servo, which is unintuitive for those that have other limit cases / output types.
### Expected or desired behaviour
1. There should be a toggle switch for reversal state
2. Motor outputs should display `MOT_PWM_MIN/MAX` instead of the servo ones (especially since at the moment the servo ones are not even considered for motors)
3. Actuator outputs should display the `ACTUATORn_INC` parameter, with the servo min/max
4. Relay outputs should show the on/off state and `RELAYn_DEFAULT` rather than servo limits and trim
- some example UI in #2766
5. Leak detector inputs should show the default level (low/high) and the pin
4. GPIO outputs (like relay / leak detector) should also display the function as `GPIO (Relay X)` (instead of Disabled / GPIO, since the `SERVOn_FUNCTION` parameter is ignored for relays)
- ideally they should also be _selectable_ as "Relay X" in the dropdown, and set the corresponding `RELAYx_PIN` value, and the `SERVOn_FUNCTION` to GPIO
- in a very ideal world they would account for external protocols like DroneCAN and allow selecting a protocol function + a GPIO configuration (set in `SERVO_GPIO_MASK`)
- Could maybe show `GPIO (??)` if there is no known function configured to use the pin
7. It would be nice if there was a space for warnings about mis-configuration, like if a leak detector is assigned to the same pin as a servo channel with some other output function (like lights, for example)
### Prerequisites
- [x] I have checked to make sure that a similar request has not already been filed or fixed.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.