Skip to content
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

A new mag calibration algorithm is needed for reducing data storage heap #8271

Closed
zarathustr opened this issue Nov 12, 2017 · 5 comments
Closed

Comments

@zarathustr
Copy link

zarathustr commented Nov 12, 2017

The magnetometer calibration is based on a least square fitting of raw data. Currently, the algorithm requires a heap load of raw data storage arrays. This point significantly increases the online memory usage of the algorithm also increases instability of managing the RAM. A better issue is raised here to integrate my recent contributions into the PX4 that it no longer stores the raw data and uses a 10x10 matrix to compute the singular value decomposition results. Such method swaps the execution time with the space consumption, which is better for low-configuration platforms e.g. stm32.

@zarathustr zarathustr changed the title A new mag calibration algorithm is need for reducing data storage heap A new mag calibration algorithm is needed for reducing data storage heap Nov 12, 2017
@dagar dagar added this to the Release v1.8.0 milestone Nov 12, 2017
@dagar
Copy link
Member

dagar commented Nov 12, 2017

Let's talk about doing this after v1.7.0 goes out shortly. Calibration code will be moving out of commander (#8200) and needs other improvements (#8265) and fixes (#8190).

@zarathustr
Copy link
Author

@dagar So when will we receive the 1.7.0 release? Then what are the main advances included in this release?

@dagar
Copy link
Member

dagar commented Nov 23, 2017

We're in the process of testing and resolving a few remaining issues. https://github.com/PX4/Firmware/milestone/23

It's a huge list of things (everything that went into master over the last 4-5 months).

@mhkabir
Copy link
Member

mhkabir commented Apr 15, 2018

@zarathustr Could you open a PR with your changes to the calibration routine?

@dagar dagar removed this from the Release v1.8.0 milestone May 4, 2018
@dagar
Copy link
Member

dagar commented Aug 21, 2020

Related, I've been playing with mag calibration via UKF here. #15059

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants