farmerbriantee / AgOpenGPS

Ag Precision Mapping, Section Control and Guidance Software
Other
641 stars 258 forks source link

IMU is always factored in even at its minimum setting #313

Closed mirh closed 1 year ago

mirh commented 1 year ago

Due to reasons, I needed to "exclude" the IMU from being taken into account for guidance... (and I just barely figured out that had to be the problem, because in my other machine without it I never had anything so blatantly off)

Yet, even with "IMU GPS Fusion" on its rightmost side, my sudden drifting/jerking problem when starting from a standstill persisted. ...which I can still guess has its sense somehow, since starting from position zero would leave the GPS heading with only a pointwise frame of reference for the first half-second or something. So even the slightest breeze of an input from the IMU would still win here.

Yet, you can see why this is a bit of an anti-feature. I mean, both because of my "particular situation", but also because logically it's kinda counter-intuitive. If the slider says "0 IMU", that should mean 0 IMU (unless, I don't know, you want to leave the task of its complete cut off without ifs or buts to some separate button). It shouldn't be that only if you neutralize its code in the Autosteer_USB ino, you can finally free yourself from its corrections.

mirh commented 1 year ago

Also tangentially related, that the IMU value (or if not any its graphical representation with the level/horizon at least) doesn't actually represent the "instantaneous signal" but just the last received value. If for whatever reason there are no more readings (because you disconnected the board, or because you just fried it entirely..) it stays stuck and doesn't reset back to 0.

richardklasens commented 1 year ago

See comment in other imu issue

mirh commented 1 year ago

I suppose I may have infringed on the "one problem, one issue" rule (you start to get depressed after a while).. but why in the heaven did you close this? IMU having no way to be removed from the equation still seems meaningful.

richardklasens commented 1 year ago

See comment in other imu issue

Because of this

mirh commented 1 year ago

That's not an explanation at all, it's just a badly formatted copy-paste of my second comment here. If you consider that to be separate from my first post, it's certainly reasonable to "move elsewhere" in a dedicated discussion.. but there's nothing in any shape or form addressing the fact that the UI doesn't have any way to completely "ditch" the IMU information.

richardklasens commented 1 year ago

You really have to do something about the way you write down your reactions. It seems you have absolutely no idea how to conversate in a normal way with a group of people who maintain this project just for learning, fun, helping farmers etc.

If you need to rip off your imu due to its bad behavior, then you need to solve the bad behavior instead of almost forcing us to create a function to turn it off completely.

For the last time.. in the coming release the problems people occur with IMU’s and driving directions, spinning tractors in the screen and such stuff should be solved.

We do not going to add an IMU OFF button for your particular problem.. solve your problem.

now I need to find how to lock this conversation because I am done with the forced language you use in your comments