Skip to content

Conversation

@Apehaenger
Copy link
Collaborator

@Apehaenger Apehaenger commented Feb 9, 2026

I do like to charge at max. possible power.
To achieve this I've to respect docks PSU current, which is on save defaults, but might be changed via PowerService-Register.
In addition I need to estimate the power which get already burned by the mower. This is mainly the current into the DCDC's and got already implemented in an earlier PR.
There remains some unmeasured (blind) power going into the ESCs which get handled (together with a measurement inaccuracy estimation) as safety-reserve.
This is how I achieved this (feel free to judge/comment if there's a better approach):

  • Add charger methods readAdapterCurrent() and setAdapterCurrent()
  • Add PowerService functions to get configured charge currents and adapter current limits
  • Add a PowerService::SetPowerManagementCallback() method so that a robot is capable to add his individual power management settings, based on his available sensors
  • Add a Sabo specific SaboRobot::OnPowerManagement() dynamic power management callback

Related PRs:
xtech/definitions-open-mower#18
ClemensElflein/open_mower_ros#267

Remark:
The BQ2579 implementation didn't got tested by me due to missing device. Someone need to check/test!

- add dynamic power management callback to adjust charger settings based on system load
- implement adapter current reading and setting for bq2576 and bq2579 chargers
- add power management logic to sabo robot for safe current allocation
- update charger driver interface with adapter current methods
- improve documentation and initialization order in sabo robot

🔧 chore(services): update submodule reference

- update services submodule to latest commit
@Apehaenger Apehaenger marked this pull request as ready for review February 9, 2026 21:21
@Apehaenger Apehaenger merged commit d723a34 into main Feb 10, 2026
5 checks passed
@Apehaenger Apehaenger deleted the feature/power-management branch February 10, 2026 21:36
Apehaenger added a commit to ClemensElflein/open_mower_ros that referenced this pull request Feb 10, 2026
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.

1 participant