Ryan Johnston
Ryan Johnston
Tested the fix. Works as intended. 👍
We also use a lion battery with higher v drop under load. Lots of logs if needed.
> I made a commit to remove the throttle based load compensation because it's bound to produce unpredictable results instead of making the state of charge estimate more accurate. >...
> > We also use a lion battery with higher v drop under load. Lots of logs if needed. > > @ryanjAA Would love to have some logs to test...
> Rebased you PR on main and did a couple of flights to completely drain a battery. Behavior was great. % only increased 2-3% after landing. > > https://review.px4.io/plot_app?log=4771723d-779d-47f4-bf61-351b8a7ca6bb https://review.px4.io/plot_app?log=80260811-cdce-40c3-a2ba-3916526964d0...
Maybe im missing something @chfriedrich98 but this log https://review.px4.io/plot_app?log=0e63b40b-adda-4a21-9b04-599d1dadbb94 says when I run the batt estimator that its soc is zero quite prematurely.
ahh it's set for lipo. ok so -e and set end voltage. @AlexKlimaj what are you cutting off at? 2.5v (per cell) is allowable but in my experience if you...
> Ideally I would like to get the parameters directly from the log s.t. you don't have to set it manually in the replay file, but I've not gotten around...
> > ahh it's set for lipo. ok so -e and set end voltage. @AlexKlimaj what are you cutting off at? 2.5v (per cell) is allowable but in my experience...
This is similar to https://github.com/PX4/PX4-Autopilot/issues/23340