i fixed the analyzer bug, when i rewrote the analyzer as a separate window, i just forgot to move the function over that actually copies the checkbox state for 'percentage' to the appropriate variable in the analyzer structure. threw in a conditional memory on that wideband/narrowband thing for you too, it clears itself if the wideband is ever disabled.
https://github.com/resfilter/eehack/...98126b2c44df6c
i glanced and i think i found the controller bug, due to thread synchronization. under the current scheme it's possible that:
- user sets the 'need to send command' flag
- event loop recognizes the flag
- event loop sends command
- user sets the 'needs to send command' flag again
- event loop goes 'ok, i've sent the command, clear the flag'
in this case your additional command happened in the middle of a send procedure and is discarded. the proper command should be sent on the next iteration anyway, but it could definitely do some weird things where your intended commands lag behind, so even though you select a 900 rpm idle, it's still at 600rpm ....
i've done a minor update to solve it in 99% of cases by clearing the flag immediately as it's taken, and when i get home i'll do some proper locking on that operation to eliminate it completely. fixed a few other functions that might be afflicted as well.
https://github.com/resfilter/eehack/...5ac0b581a31a94
Bookmarks