Refloat: restore config before locking (and writing config) #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A carbon copy of the Float package from the v1.3 commit: b2c2750 Except for paths and Makefiles adjusted to the new name, and a few auto-generated files omitted.
lefthook is a git hook manager.
Remove useless comments and move long trailing comments that mess up clang-format.
Doesn't make sense to call fabsf() on a boolean, assume the intention was to take absolute value of the tiltconst variable.
Using qmlformat and a lot of hand-editing where it gave suboptimal results.
Signed-off-by: Dado Mista <dadomista@gmail.com>
When going backwards (battery forward), the sign for the accel_gap is inverted. Without the fix, the wrong ATR strength was being applied, so e.g. setting only ATR uphill, the nose would instead angle down on a downhill if going battery forward.
From AppUI (or Float Watch) it was possible to enter flywheel mode without actually passing the sanity check! Signed-off-by: Dado Mista <dadomista@gmail.com>
Signed-off-by: Dado Mista <dadomista@gmail.com>
Check against roll tolerances when starting dark ride Signed-off-by: Dado Mista <dadomista@gmail.com> darkfix
Reverse stop condition now also checked in wheelslip state! Signed-off-by: Dado Mista <dadomista@gmail.com>
200ms instead of 500ms Signed-off-by: Dado Mista <dadomista@gmail.com>
Drop the version in settings.xml, it adds more complexity than it's worth. Use semantic versioning and make version a string.
The simplestart/posi features shouldn't override the footpad sensor state to FS_BOTH, as that should represent the true value of the sensors. Instead, add a function that evaluates whether the sensors are engaged and which takes simplestart and posi into account. One of the practical advantages is the user is now able to see the true state of the sensors when only one side is activated, as opposed to always seeing both sides are.
Keep board explicitly engaged in flywheel mode instead of overriding float_conf adc thresholds.
Do the evalution in footpad_sensor_update().
Also fix atr_filter_enabled config value being reset in an init function. The _reset name is much more descriptive, the function serve to reset any runtime values that haven't been cleared after stopping (so that the values remain available for backward reference).
Fix: Fix duty on LED status bar always showing at 100%.
Fix: Fix disappearing battery percentage on LED status bar.
Fix: Fix LEDs configuration not picking up some values after startup.
Fix: Rename the Specs -> LED Pin option values to be more descriptive.
Convert brightness from 0..1 float to 0..100 integer Fix: Fix setting brightness for LCM module (Floatwheel) Signed-off-by: Dado Mista <dadomista@gmail.com>
Fix: Fix Torque Tilt using ATR step sizes instead of its own step sizes.
Fix: Fix showing wheelslip and duty pushback alerts in Flywheel mode.
Fix: Fix beeper beeping on board startup.
Signed-off-by: Dado Mista <dadomista@gmail.com>
The descriptions were outdated.
Fix: Fix a crash in LEDs Cipher transition when front or rear strips have 0 LEDs configured.
Fix: Fix a (Watchdog Timer) reboot when reinstalling Refloat (in case LEDs are disabled)
Locking the board via 3rd party apps while still in flywheel mode could cause the temporary flywheel changes to be made permanent. Now the lock command first reads the config then sets the package disabled flag and writes the config. Signed-off-by: Dado Mista <dadomista@gmail.com>
|
You did the fix in Float, not in Refloat 😅 You might wanna wait for the repo move (which I hope will be finished in a week or two, I'm still swamped time-wise), then it'll be a bit simpler :) |
|
lmao, sorry - I did it in both but somehow still picked the wrong commit to PR |
|
😁 please don't add it here anymore, I've got the new repo all ready, but didn't have time for the actual switch. I'm leaving for the weekend, when I get back I'll do it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Locking the board via 3rd party apps while still in flywheel mode could cause the temporary flywheel changes to be made permanent. Now the lock command first reads the config then sets the package disabled flag and writes the config.