Skip to content

Commit

Permalink
Fix SBUS2 sensor units; use headspeed instead of eRPM (#135)
Browse files Browse the repository at this point in the history
* Fix temperatur unit

* Fix some units

Temperatures and pwm were reading wrong.

Change rpm source to match rpm sensor, as erpm didn't seem to match actual headspeed in my tests with hobbywing v4 based esc. (align 300x combo esc)

* Add note about gear ratio in RPM and Kontronik ESC sensors
  • Loading branch information
mmosca authored Aug 15, 2024
1 parent 81666ca commit d4c2929
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/Telemetry.md
Original file line number Diff line number Diff line change
Expand Up @@ -345,7 +345,7 @@ It runs at a fixed baud rate of 100000 8e2.
```

### Radio Configuration
The following sensors should setup on your radio
The following sensors should setup on your radio and Gear ratio for RPM Sensor and Kontronik ESC to 1.00 in your radio.
| Slot | Sensort Type | Notes |
| --- | --- | --- |
| 1 | Voltage | FC Voltage sensor. Pack and cell voltages |
Expand Down
4 changes: 2 additions & 2 deletions src/main/telemetry/sbus2.c
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ void handleSbus2Telemetry(timeUs_t currentTimeUs)

if (escData != NULL) {
// 8 slots, esc
send_kontronik(8, escData->voltage * 0.1f, escData->consumption * 100, escData->erpm, escData->current * 0.01f , escData->temperature * 10, escData->temperature2 * 10, escData->bec_current * 10, escData->pwm);
send_kontronik(8, escData->voltage * 0.1f, escData->consumption * 100, rpm, escData->current * 0.01f, escData->temperature * 0.1f, escData->temperature2 * 0.1f, escData->bec_current * 10, escData->pwm * 0.1f);
}
}

Expand Down Expand Up @@ -160,4 +160,4 @@ FAST_CODE void taskSendSbus2Telemetry(timeUs_t currentTimeUs)
}
}

#endif
#endif

0 comments on commit d4c2929

Please sign in to comment.