-
Notifications
You must be signed in to change notification settings - Fork 102
Added UserText OSD support on Rapidfire Module #179
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
base: master
Are you sure you want to change the base?
Conversation
I'm under the impression that the custom osd has been broken... since forever. When was this fixed or what has changed? |
Dunno, the SPI documentation was a bit off as well. I am using Rapidfire Brain v1.3.3. (Latest) |
17a1e33
to
8751c14
Compare
Hey just wondering if you have to downgrade RadpidFire FW to a specific version or it works even with the latest FW? |
Hey @cnheider , I've just tested your PR on my HDZero + Rapidfire + EP82. It works, I can see messaged! Great result! However there are a few issues:
If it is the case that Rapidfire can only show a single line, then perhaps we should display only those ones that are important. I'd say that would be your laptime. Or maybe somehow format them in one line? |
I also reworked the Rapidfire SPI implementation using the excellent work done by GOROman
https://github.com/GOROman/rapidFIRE_SPI
I implemented OSD support for the Rapidfire Module in ELRS Backpack and verified that it works with RotorHazard.
https://github.com/i-am-grub/VRxC_ELRS
I will post some screenshots later.