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

Signal K support for extra fields from Battery Configuration Status #578

Open
fabdrol opened this issue Jul 15, 2020 · 4 comments
Open

Signal K support for extra fields from Battery Configuration Status #578

fabdrol opened this issue Jul 15, 2020 · 4 comments

Comments

@fabdrol
Copy link
Member

fabdrol commented Jul 15, 2020

Recently I needed to represent additional battery information from PGN 127513 (Battery Configuration Status). Some of these fields are not yet present. This is how the canboat output looks:

const ex = {
   "pgn":127513,
   "description":"Battery Configuration Status",
   "fields":{
      "Instance":16,
      "Battery Type":"Gel", // electrical.batteries.16.chemistry
      "Supports Equalization":0, // NOT PRESENT IN SK
      "Nominal Voltage":"12", // NOT PRESENT IN SK
      "Capacity":0, // electrical.batteries.16.capacity.nominal
      "Temperature Coefficient":19, // NOT PRESENT IN SK
      "Peukert Exponent":0.002, // NOT PRESENT IN SK
      "Charge Efficiency Factor":0, // NOT PRESENT IN SK
   }
}

I would like to propose the following fields. If nobody objects, I'll do PRs for n2k-signalk and specification

  • electrical.batteries.<inst>.voltage.nominal
  • electrical.batteries.<inst>.temperature.coefficient
  • electrical.batteries.<inst>.peukertExponent
  • electrical.batteries.<inst>.chargeEfficiency
  • electrical.batteries.<inst>.supportsEqualisation

Thoughts?

@fabdrol fabdrol self-assigned this Jul 15, 2020
@fabdrol fabdrol changed the title Additional battery fields Signal K support for Battery Configuration Status Aug 7, 2020
@fabdrol fabdrol changed the title Signal K support for Battery Configuration Status Signal K support for extra fields from Battery Configuration Status Aug 7, 2020
@natecostello
Copy link

Has there been any consideration of adding data for individual cells (voltage and temperature)? This is important for DIY battery systems. Several BMS provide this data over CAN and it would be great to be able to capture it with Signal K.

@tkurki
Copy link
Member

tkurki commented Dec 25, 2020

@natecostello I just created #603 for that.

@tkurki
Copy link
Member

tkurki commented Dec 25, 2020

@fabdrol I have completely missed this issue, makes sense to add that.

What is the temperature coefficient? I would put that as temperatureCoefficient, as it is not really a "temperature".

@GHBLoos
Copy link

GHBLoos commented Apr 1, 2021

LifePo4 batteries do not have to be equalised, but some do have balancing.
electrical.batteries.<inst>.supportsBalancing = true, false, auto ?

Just having the support of equalisation or balancing may not be enough information. Maybe something must be said about the status:
isBalancing
isEqualizing
Balancing = on, off
Equalizing = on, off

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants