-
Notifications
You must be signed in to change notification settings - Fork 6
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
Thomas/add hardware support #9
base: master
Are you sure you want to change the base?
Conversation
…0 for now since it isnt used. Remove some annoying logging. Clean up some files.
…he now unnecessary flight modules. Converted everything to f64." This reverts commit b4c5a63.
Is it correctly holding position, but just noisy? |
Nope. Moving randomly.
…On Tue, Jan 16, 2018 at 10:15 AM Martin Deegan ***@***.***> wrote:
Is it correctly holding position, but just noisy?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABOrXPp5Le5aSP5j8Tr69WlTxl9QyuvFks5tLOb3gaJpZM4RfaN->
.
|
Actually it looks fine I think: It starts on the opposite side then converges over to the correct side... |
Update this when you have correctly calibrated the sensors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before we add this, it might be a good idea to also implement the software PWM struct as well as making it modular.
This did fix the Kalman filter. |
Reverts the problematic "Removed som random matlab generated files..." commit but it doesn't seem to help with the filter. Looks weirdly noisy to me. We can remove this revert commit from the pull request if we don't think that was actually the problematic commit.
Add support for my drone hardware.