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

Show system battery as first battery #368

Closed
martinbosma opened this issue May 3, 2024 · 2 comments
Closed

Show system battery as first battery #368

martinbosma opened this issue May 3, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@martinbosma
Copy link
Contributor

martinbosma commented May 3, 2024

The system battery should be shown as first battery.
The battery data is published on system/0/Batteries as a json object.
The system battery can be recognized by the active_battery_service property set to true.

Below example shows a list of Lynx Smart BMSes, which are combined into a new 'virtual' parallel BMS, which is used as system battery.

{
  "value": [
    {
      "active_battery_service": false,
      "current": 0,
      "id": "com.victronenergy.battery.socketcan_can0_vi3_uc314664",
      "instance": 3,
      "name": "Lynx Smart BMS Bottom",
      "power": 0,
      "soc": 99.98999786376953,
      "state": 0,
      "voltage": 13.5
    },
    {
      "active_battery_service": false,
      "current": 0,
      "id": "com.victronenergy.battery.socketcan_can0_vi2_uc314657",
      "instance": 2,
      "name": "Lynx Smart BMS Top",
      "power": 0,
      "soc": 98.37999725341797,
      "state": 0,
      "voltage": 13.510000228881836
    },
    {
      "active_battery_service": true,
      "current": 0,
      "id": "com.victronenergy.battery.lynxparallel0",
      "instance": 4,
      "name": "Lynx Smart BMS Totals",
      "power": 0,
      "soc": 99.18499755859375,
      "state": 0,
      "timetogo": 864000,
      "voltage": 13.505000114440918
    }
  ]
}
@mman
Copy link
Collaborator

mman commented May 22, 2024

@martinbosma What would be the easiest way to simulate such system? The fix looks easy enough but I need a way to verify it works as expected.

I'm looking at https://github.com/victronenergy/venus-docker and it does not seem to contain any pre-recorded Lynx BMS configs, or does it?

@mman mman added the enhancement New feature or request label Jun 20, 2024
@mman
Copy link
Collaborator

mman commented Jul 22, 2024

Fixed via #371

@mman mman closed this as completed Jul 22, 2024
@martinbosma martinbosma added this to the v3.50 milestone Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants