-
Notifications
You must be signed in to change notification settings - Fork 0
SBG Driver #3
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: main
Are you sure you want to change the base?
SBG Driver #3
Conversation
|
Can you add this to the workspace Cargo.toml. TLDR: Optimisation makes you fall of the trampoline and hurt yourself 🤕 I say it this way because I'm not entirely sure why this happens with sbg-rs with any other opt-level, but you get a trampoline related failure. [profile.release.package.sbg-rs] |
|
I think the build fails because of this message repo being updated https://github.com/uorocketry/messages Check your lock 🔒 |
We want debug true for release too? |
Oops, I forgot about that... I may have run |
I don't think it's needed 🤔 I believe I just never removed it from my search into what was going wrong. |
|
@NoahSprenger can you take a look at fixing some of these errors. I haven't been able to figure out some of the updates that you made to the messages crate which makes this very awkward to find the actual structs or enums that are expected. There are some other issues with what is a RadioMessage vs a CanMessage, and also the lifetime specifiers on all the radio stuff are a pain to deal with. |
Yep, I can give you a hand. Can you elaborate on what you by "There are some other issues with what is a RadioMessage vs a CanMessage". |
There are places where something is expecting a RadioMessage or a CanMessage and receiving the other. Specifically some stuff with Madgwick I think where the RTIC task passes in a CanMessage, but it's operating on SBG data from the RadioMessage. |
Added the sbg-rs crate which provides a Rust wrapper around the sbgECom C library.
A custom build script compiles and links the library and uses bindgen to generate a bindings file for given library headers.
TODO: