Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log reason for docking. Allow for short voltage drops during mowing due to current spikes. #82

Merged
merged 2 commits into from
May 31, 2024

Conversation

olliewalsh
Copy link
Collaborator

@olliewalsh olliewalsh commented May 3, 2024

If voltage drops and remains below empty voltage for 20s then dock.
If voltage drops below critical voltage immediate dock.
OM_BATTERY_CRITICAL_VOLTAGE (the related param of it) defaults to OM_BATTERY_EMPTY_VOLTAGE if not set, preserving current behaviour
Lowered default empty voltage to 24V,, at 25V almost half of the battery capacity remains.
On my c500 24V still leaves at least 30mins of time if stuck in emergency before hitting BMS undervoltage which seems to be around 19.5-20.0V.

@olliewalsh olliewalsh changed the title Log dock reason and only abort when mowing Log dock reason and allow for voltage droop in battery low logic May 30, 2024
@ClemensElflein ClemensElflein changed the title Log dock reason and allow for voltage droop in battery low logic Log reason for docking. Allow for short voltage drops during mowing due to current spikes. May 31, 2024
@ClemensElflein ClemensElflein merged commit a178063 into ClemensElflein:main May 31, 2024
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants