infineon4engineers Facebook

infineon@google+ Google+

infineon@linkedin linkedin

infineon4engi@twitter twitter

infineon@youtube youtube

+ Reply to Thread
Results 1 to 6 of 6

Thread: Potential Error in the new Copter Rocks software

  1. #1
    New Member New Member Marcell is on a distinguished road
    Join Date
    Jun 2016
    Location
    Graz
    Posts
    3
    Points
    40

    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.

  2. #2
    New Member New Member
    Infineon Employee
    Infineon Employee
    Rafael Gattringer is on a distinguished road
    Join Date
    Dec 2015
    Location
    Vienna
    Posts
    13
    Points
    108.75
    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.

  3. #3
    New Member New Member
    Infineon Employee
    Infineon Employee
    cherry is on a distinguished road
    Join Date
    Sep 2016
    Posts
    1
    Points
    20
    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.

  4. #4
    New Member New Member Marcell is on a distinguished road
    Join Date
    Jun 2016
    Location
    Graz
    Posts
    3
    Points
    40
    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)

  5. #5
    New Member New Member
    Infineon Employee
    Infineon Employee
    Rafael Gattringer is on a distinguished road
    Join Date
    Dec 2015
    Location
    Vienna
    Posts
    13
    Points
    108.75
    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.

  6. #6
    New Member New Member Marcell is on a distinguished road
    Join Date
    Jun 2016
    Location
    Graz
    Posts
    3
    Points
    40
    This file: https://github.com/ManagementCenterI...9X50/MPU9150.c

    I made the changes and sent a pull request on github.

+ Reply to Thread
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.