Changes with R1625 introduced an error to MS561101 code

This forum is dedicated to software development related to MultiWii.
It is not the right place to submit a setup problem.
Software download
Post Reply
User avatar
EOSBandi
Posts: 802
Joined: Sun Jun 19, 2011 11:32 am
Location: Budapest, Hungary
Contact:

Changes with R1625 introduced an error to MS561101 code

Post by EOSBandi »

Hi !
It seems that changes made to MS561101 calculations introduced an error. Under 20 degree Celsius, the altitude value will go haywire. Apparently changing to float from int64_t introduced an overflow somewhere (BaroTemp value will be nonsense).
As a quick fix I reverted these changes in my navi trunk, but since baro code does not belong to me, I would rather not touch it in the _shared.

Y.Mita
Posts: 46
Joined: Thu Sep 15, 2011 11:25 pm

Re: Changes with R1625 introduced an error to MS561101 code

Post by Y.Mita »

Same at my copter.
Sensor is Drotek 10DOF MS.
Before version up, stable under +-0.2m. After, +-3.0m.
Check twice on two configuration.

User avatar
EOSBandi
Posts: 802
Joined: Sun Jun 19, 2011 11:32 am
Location: Budapest, Hungary
Contact:

Re: Changes with R1625 introduced an error to MS561101 code

Post by EOSBandi »

Y.Mita wrote:Same at my copter.
Sensor is Drotek 10DOF MS.
Before version up, stable under +-0.2m. After, +-3.0m.
Check twice on two configuration.

It even worse, since it will gain or loose altitude constantly (at least the baro will think...)

Alexinparis
Posts: 1630
Joined: Wed Jan 19, 2011 9:07 pm

Re: Changes with R1625 introduced an error to MS561101 code

Post by Alexinparis »

Sorry for this.

I think I found the bug (tested on the bench with some ice ;) ).
and corrected in r1649

Y.Mita
Posts: 46
Joined: Thu Sep 15, 2011 11:25 pm

Re: Changes with R1625 introduced an error to MS561101 code

Post by Y.Mita »

Alexinparis wrote:Sorry for this.

I think I found the bug (tested on the bench with some ice ;) ).
and corrected in r1649

Thank you!
I tried it on desktop, and seems more stable than before!
Tomorrow, I'll try it at field with MWNavi.

Y.Mita
Posts: 46
Joined: Thu Sep 15, 2011 11:25 pm

Re: Changes with R1625 introduced an error to MS561101 code

Post by Y.Mita »

Y.Mita wrote:
Alexinparis wrote:Sorry for this.

I think I found the bug (tested on the bench with some ice ;) ).
and corrected in r1649

Thank you!
I tried it on desktop, and seems more stable than before!
Tomorrow, I'll try it at field with MWNavi.


Yesterday, I went field and test flight my copter.
No problem, and stable than "MultiWii_2.3-navi-b4-xmas".
I replaced Sensor.cpp of "MultiWi2_3-navi-b5".

Thank you!

Post Reply