search

Zubax / gridsens_gs2

Zubax GNSS module
https://zubax.com/products/gnss_2
GNU General Public License v3.0
52 stars 57 forks source link

Control the nav_filter model with a parameter #9

Closed WickedShell closed 7 years ago

WickedShell commented 8 years ago

Also updates default value of the nav_filter from Airborne_2g to Airborne_4g. (APM testing shows that Airborne_4g has less filtering on the data, and the minimum amount of filtering is desired).

Also updates the list of DynModels to acknowledge that 9 is the dynamic model for wrists, its only used in newer GPS firmware, but the GPS chip can be upgraded to that firmware.

pavel-kirienko commented 7 years ago

Hey @WickedShell! Could you please review my feedback? We're releasing firmware version 4 soon, would be great to merge your changes in too.

WickedShell commented 7 years ago

Sorry missed the most recent round of comments in my inbox. I'll try and get some time allocated within the near future to finish fixing this up. (I encountered some problems with my Zubax that I suspect are cabling but haven't been able to debug it yet. My babel came in so hopefully with that I can get it going again).

pavel-kirienko commented 7 years ago

No worries. We have time until late April to merge this and ship with the newest batch of hardware ;)

pavel-kirienko commented 7 years ago

A cool bootloader is coming up too, supports USB, UART, and CAN, so upgrading in the field will be much easier.

WickedShell commented 7 years ago

@pavel-kirienko I pushed fixes (apparently I'd actually done 90% of the requested changes). Due to a slight cabling problem here I haven't flashed it yet to validate that it all still behaves correctly, but wanted to get it pushed up first, so it stopped languishing. Will hopefully find the needed cables (and time) to validate this the rest of the way tomorrow.

pavel-kirienko commented 7 years ago

@WickedShell should we still keep this open?