Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • The error returns are very simple and should be changed to enums that provide more info.

  • The sensor fusion frequency constant could be defined in a better location for organization.

  • MadGWick algorithm did not work as expected. Find new one

  • In the SF_GetPosition function, there are many arrays that have not been initialized on declaration.

  • See if we can abstract the GPS coordinate conversion methods so both waypoint and SF can use them

Safety

It’s just weird. Bad function names, bad comments, redundant functions… There’s not that much of it, but it’d be realllly nice to go thru the whole thing and clean up.

...

  • We need to rename the functions such as `SendFromPMToAM()`. These names are hard to understand

GPS Sensor

  • Use

Styling

  • Style guide

    • Consistent function naming scheme: are we capitalizing the first letter or not?

    • Consistent file naming scheme: are we capitalizing the first letter or not?