Bob Long
Bob Long
Basically, as soon as you land, and get a prearm failure for still being in the landing sequence, you'll briefly get something like "accels unhealthy" or "IMU temperature low" on...
Here's a tlog I created so you can reproduce the behavior I'm talking about and test that the PR fixes it. Right after landing, the HUD pops up a "PreArm:...
@meee1 any thoughts on this?
~~Ah crap, this doesn't work. I'm passing current throttle in to the idle governor function, when it needs me to pass min throttle~~ Fixed, but overall, my approach is a...
> Can we get it to the existing throttle override stuff here?: Ah, great call. Yeah, that's where it belongs. > I think a longer term fix would be to...
I've moved it. Thanks @IamPete1 , this is way better. I think the only reason this wasn't originally handled in `throttle_override` was that there used to be an early-return when...
> I really like the cleanup. The only disadvantage I see is that we miss telling TECS if we bump the min throttle. Hmm, good point. It's probably not the...
Just pushed something that I _think_ works, but it's pretty fresh and I haven't tested yet. As a bonus, we're now letting the TECS know when the redline governor is...
One little kink to work out, the TECS min/max throttle notification stuff causes a sawtooth instead of what is intended. This has nothing to do with the existing lines 599...
Okay, another little rework. 1. I've abandoned informing the TECS about redline governing. The redline governor is a bit of a mess, and unlike the idle governor, it chucks out...