Comments (9)
I found the root cause of the problem!
I was right. I am missing an entry in the deviceNames[DEVBOARD_COUNT]
array.
In the screencap below, at line 184
, there is an index entry for the STM32F405RG in the ct_devboards_e
enumerator.
Then, in the deviceNames[DEVBOARD_COUNT]
array, the entry for the STM32F405RG is missing at line 368
.
Therefore, everything from the STM32F407VE entry all the way up to the Teensy 4.1 is "shifted down" one index value.
But, the index value of the STM32F401RE and below, the index value provided by the call to device.type.devboard
is correct.
from crsfforarduino.
Nice! I'm happy it was an easy fix, and glad I could help! Thanks for this awesome lib, it saved me so much time!
from crsfforarduino.
Ok I'll go with option B, it's working on my end, hopefully no one will run into this issue in the meantime. Have a fun break!
Thanks
from crsfforarduino.
Taking a deeper look the const char *CompatibilityTable::getDevboardName() function in the compatibility table was returning an empty string. When I changed line 717 from return deviceNames[device.type.devboard]; to return deviceNames[device.type.devboard - 1]; I'm now getting "teensy4.1" returned as I should and everything works.
from crsfforarduino.
Taking a deeper look the const char *CompatibilityTable::getDevboardName() function in the compatibility table was returning an empty string. When I changed line 717 from return deviceNames[device.type.devboard]; to return deviceNames[device.type.devboard - 1]; I'm now getting "teensy4.1" returned as I should and everything works.
That tells me I am missing an entry in the Compatibility Table, specifically with the deviceNames[DEVBOARD_COUNT]
array.
Fortunately, that's an easy fix; and it can be pushed out in the next patch release to v1.0.x.
Thank you very much for your work with this. You have saved me the time in figuring this one out, and now I can release a fix for this almost straight away.
I say "almost", because I am currently on hiatus until the end of the first week of April of 2024.
from crsfforarduino.
You're very welcome. =^/.^=
If you're able, can you fork CRSFforArduino (if you have not done so already), and submit a Pull Request back to the upstream Version-1.0.x-Maintenance-Branch with the fix that's outlined here?
If you're doing the Pull Request thing, you MUST
enable Commit Signing with either a GPG Key or an SSH Key before the first commit in your Pull Request.
If you already have this enabled, your commits will show up as a green Verified
tag next to each individual commit. Otherwise, it your commits will show up as a yellow Unverified
tag next to the individual commit.
Due to my branch protection rules, I simply cannot accept Pull Requests with Unverified
commits.
from crsfforarduino.
Will this just give me credit for the change? I don't need credit but if there's some other reason that you want me to do this let me know. I'm new to github and don't know how any of the commit signing or GPG key thing works. Thanks!
from crsfforarduino.
Will this just give me credit for the change? I don't need credit but if there's some other reason that you want me to do this let me know. I'm new to github and don't know how any of the commit signing or GPG key thing works. Thanks!
It's all good. =^/.^=
The only reason why I asked is 'cause I'm currently taking a break from this all, and if you wanted the fix done straight away, I can merge Pull Requests remotely.
I am able to fix this on my end, but it will need to wait until my break is over.
So, it's simply a question of "how quickly would you like this done"?
Option A: Add this yourself, and submit a Pull Request back to me. Then, I can cut-and-release v1.0.3 immediately after I merge the Pull Request, provided all of the checks (barring the Arduino IDE build check, 'cause I am currently having ongoing problems with that one) in the Pull Request pass.
Or...
Option B: Wait for me to start back on this on 2024-4-7 NZDT, and I will cut-and-release this fix as v1.0.3 within that week.
PS: Yes, people that submit Pull Requests are credited as the contributors to CRSF for Arduino; and the project itself relies on its user-base to test and debug the code-base, because I simply don't have the luxury of having access to all the 150+ hardware targets that CRSF for Arduino is compatible with.
...and so far, this process works. =^/,..,^=
Anyone is welcome to contribute where and when they are able. Even doing this counts.
But, it's the Pull Requests what get your name listed as an official contributor to the project.
from crsfforarduino.
I will. =^/,..,^=
from crsfforarduino.
Related Issues (20)
- CRSF frame parsing timeout repeatedly expires thus dropping all frames HOT 5
- Raspberry Pi Pico W HOT 5
- Version 1.1.0 Development Tracking HOT 2
- Read CRSF Telemetry Mirrored to Radio Aux Ports HOT 1
- Docs guidance on how to wiring up a receiver. HOT 3
- Replace existing GNU GPL v3 licence with GNU Aferro GPL v3.
- Add custom baud rate for CRSF protocol HOT 1
- Can I communicate and interact with TX? HOT 3
- Version 1.0.0 Development Tracking HOT 1
- Seeed Studio XIAO nRF52840 compatibility HOT 5
- Refactor Compatibility Table
- Add failsafe handling
- Re-factor UART pin assignments.
- Process link statistics from incoming packets.
- Automatic UART baud-rate configuration
- No such file or directory when compiling with PlatformIO and VsCode HOT 9
- writeRcChannel - Control CRSF Transmitters with CRSFforArduino HOT 9
- Arduino IDE builds are broken
- Example CRSFforArduino example "rc_channels" not working, Library not installing correctly, no simple way to enter or change Rx/Tx pin numbers, not enough validation or testing before releasing.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from crsfforarduino.