bluerobotics / bluerobotics/BlueOS

core: frontend: vehicle-setup: reflect output pin function

Open
#3,267 2 comments 0 reactions 0 assignees View on GitHub
core enhancement triage ui
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.