infineon4engineers
infineon@google+
infineon@linkedin
infineon4engi@twitter
infineon@youtube
-
Sep 16th, 2016 06:00 AM
#1
New Member
Potential Error in the new Copter Rocks software
Hello Copter Rocks Community,
reading through the new Dave 4 Copter Rocks Software I noticed a potential error in the AttitudeController and MPU9150 files, if the sensor fusion algorithm is changed.
Code:
#define ATTITUDEALGORITHM 2
If the define is set to 2 (QuaternionFilter), the name of the GetAngles() function is misleading as it returns the Pitch and Roll Angle of the copter but for Yaw it returns its Yawrate.
However if the ATTITUDEALGORITHM 1 (Kalman filter) is used, the function returns the Roll, Pitch and Yaw angle of the copter.
So changing the attitude algorithm would cause problems in the AngleRateController as this function expects the Yawrate.
I would make a Pull request on Github to fix this but the new Software is only uploaded as zip file.
-
Sep 16th, 2016 09:41 AM
#2
Hi Marcell,
Walter will extract and reupload the single files on Monday.
Can anyone confirm this fix?
Best regards, Rafael
The views expressed here are my personal opinions, have not been reviewed or authorized by Infineon and do not necessarily represent the views of Infineon.
-
Sep 19th, 2016 12:23 AM
#3
Hello Marcell,
The code should fly as it is right now on our GitHub repository: https://github.com/ManagementCenterI...k/copter.rocks
I also unpacked the .zip file on the repository.
Please flash the software and try to get your quadcopter flying. If you have any more queries please just ask.
Best regards, Walter
The views expressed here are my personal opinions, have not been reviewed or authorized by Infineon and do not necessarily represent the views of Infineon.
-
Sep 19th, 2016 03:28 AM
#4
New Member
Hello Walter,
I just wanted to point out that there is a potential error in the code if someone changes the define for the sensor fusion algorithm. (#define ATTITUDEALGORITHM)
-
Sep 19th, 2016 04:14 AM
#5
This seems to be the relevant file on GitHub - correct?
https://github.com/ManagementCenterI...9X50/MPU9150.h
The views expressed here are my personal opinions, have not been reviewed or authorized by Infineon and do not necessarily represent the views of Infineon.
-
Sep 19th, 2016 04:56 AM
#6
New Member
This file: https://github.com/ManagementCenterI...9X50/MPU9150.c
I made the changes and sent a pull request on github.
Disclaimer
All content and materials on this site are provided “as is“. Infineon makes no warranties or
representations with regard to this content and these materials of any kind, whether express or
implied, including without limitation, warranties or representations of merchantability, fitness for
a particular purpose, title and non-infringement of any third party intellectual property right. No
license, whether express or implied, is granted by Infineon. Use of the information on this site may
require a license from a third party, or a license from Infineon.
Infineon accepts no liability for the content and materials on this site being accurate, complete or up-
to-date or for the contents of external links. Infineon distances itself expressly from the contents of
the linked pages, over the structure of which Infineon has no control.
Content on this site may contain or be subject to specific guidelines or limitations on use. All postings
and use of the content on this site are subject to the Usage Terms of the site; third parties using
this content agree to abide by any limitations or guidelines and to comply with the Usage Terms of
this site. Infineon reserves the right to make corrections, deletions, modifications, enhancements,
improvements and other changes to the content and materials, its products, programs and services
at any time or to move or discontinue any content, products, programs, or services without notice.
Bookmarks