Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: process RC packets back from CRSF module as trainer input #5724

Merged
merged 3 commits into from
Jan 3, 2025

Conversation

pkendall64
Copy link
Contributor

@pkendall64 pkendall64 commented Dec 19, 2024

Summary of changes:
Adds processing of RC Packets sent as telemetry input from CRSF (ExpressLRS) module as trainer port data.
Theres a matching PR in the ExpressLRS repo that allows the backpack head tracker data to be sent back to EdgeTX as telemetry ExpressLRS/ExpressLRS#3076.

IMG_0971 Large
IMG_0972 Large
IMG_0973 Large

@raphaelcoeffic
Copy link
Member

@pfeerick LGTM. If you're ok with it as well, it is a keeper!

@pfeerick pfeerick added this to the 2.11 milestone Dec 22, 2024
@pfeerick pfeerick added the enhancement ✨ New feature or request label Dec 22, 2024
@pfeerick
Copy link
Member

pfeerick commented Jan 2, 2025

Just finished updating the backpack on my HDZero googles and T12-Max TX/ELRS firmware and this is working perfectly. And nicely steps around a annoyance I was about to have with adding a swappable pan/tilt setup to a couple of RC cars... and needing to use a second controller with some of them.

@pfeerick pfeerick changed the title Process RC packets back from CRSF module as trainer input feat: process RC packets back from CRSF module as trainer input Jan 3, 2025
@pkendall64
Copy link
Contributor Author

Just finished updating the backpack on my HDZero googles and T12-Max TX/ELRS firmware and this is working perfectly. And nicely steps around a annoyance I was about to have with adding a swappable pan/tilt setup to a couple of RC cars... and needing to use a second controller with some of them.

Yeah, I also think you get a lot more control from EdgeTX with the mixer than trying to implement all that in the backpack!

@pfeerick pfeerick merged commit 574f753 into EdgeTX:main Jan 3, 2025
52 checks passed
@pfeerick
Copy link
Member

pfeerick commented Jan 3, 2025

Yeah, not kidding! Better to let the mixer do all the hard work for you... no need to reinvent the wheel 🤭 Now... I wonder who will be the first to try head tracker direct craft control since that is now also doable... 🤣

@ProfDrYoMan
Copy link

Hi, is there a link to some more documentation. It seems I am missing some points, but it sounds cool.

My setup right now is sending data from a head tracker via a ELRS RX (EP2, programmed as TX) over to a ELRS RX (EP2, programmed as RX) which is connected to an UART on the TX16s.

For me this PR sounds like I can remove the second ELRS RX inside the TX16s.

@pfeerick
Copy link
Member

pfeerick commented Feb 24, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants