[FEMS] Cell Voltage in V nicht mV?

TheSerapher · January 19, 2025, 4:24pm

Hey Forum,

Ich habe mal per API die Zellendaten angesehen und festgestellt das jede Zelle mit 3200V in der API angegeben ist. Das klingt viel zu hoch und ich vermute das die Einheit hier mV sein sollte? Kann mir das jemand bestätigen?

Sn0w3y · January 19, 2025, 4:33pm

Hallo,

wie liest du denn die Zellspannung aus?

Grüße !

TheSerapher · January 19, 2025, 4:35pm

  {
    "address": "battery0/Tower0Module2Cell009Voltage",
    "type": "INTEGER",
    "accessMode": "RO",
    "text": "",
    "unit": "V",
    "value": 3308
  },

Das ist die API response per REST call.

Sn0w3y · January 19, 2025, 4:48pm

Laut Code passt das so:

github.com

OpenEMS/openems/blob/79650cb4096775798ddb0c72ef50f23f99790e78/io.openems.edge.battery.fenecon.home/src/io/openems/edge/battery/fenecon/home/BatteryFeneconHomeImpl.java#L815


      
          					 *
          					 * Dynamically generate Channels and Modbus mappings for Cell-Voltages.
          					 * Channel-IDs are like "TOWER_0_MODULE_2_CELL_001_VOLTAGE".
          					 */
          					var ameVolt = new ModbusElement[cellsPerModule];
          					for (var cell = 0; cell < cellsPerModule; cell++) {
          
          						// Create Voltage Channel
          						var channelId = new ChannelIdImpl(//
          								generateSingleCellPrefix(tower, module, cell) + "_VOLTAGE",
          								Doc.of(OpenemsType.INTEGER).unit(Unit.VOLT));
          						this.addChannel(channelId);
          
          						// Create Modbus-Mapping for Voltages
          						var uwe = new UnsignedWordElement(moduleOffset + module * 100 + 2 + cell);
          						ameVolt[cell] = m(channelId, uwe);
          					}
          
          					/*
          					 * Dynamically generate Channels and Modbus mappings for temperature sensors.
          					 * Channel-IDs are like "TOWER_0_MODULE_2_TEMPERATURE_SENSOR_1".

gehe aber auch davon aus, dass hier MILLIVOLT gemeint ist @stefan.feilmeier

TheSerapher · January 19, 2025, 4:50pm

Korrekt, dieser Code sagt es sogar eindeutig was eine Zelle haben sollte

github.com

OpenEMS/openems/blob/79650cb4096775798ddb0c72ef50f23f99790e78/io.openems.edge.battery.fenecon.home/src/io/openems/edge/battery/fenecon/home/BatteryFeneconHomeHardwareType.java#L20


      
          	BATTERY_64(64, "Fenecon Home Battery 64,4Ah", 2800, 40.6f, 49.7f, 14, 5, new FeneconHomeBatteryProtection64(),
          			"519100001254", "519110001918"); //
          
          	/**
          	 * Defaults to {@link #BATTERY_52} to avoid detection failure with old firmware
          	 * versions.
          	 */
          	public static final BatteryFeneconHomeHardwareType DEFAULT = BATTERY_52;
          
          	public final int capacityPerModule; // [Wh]
          	public final float moduleMinVoltage; // [V]; e.g. 3.0 V x 14 Cells per Module
          	public final float moduleMaxVoltage; // [V]; e.g. 3.5 V x 14 Cells per Module
          	public final int cellsPerModule;
          	public final int tempSensorsPerModule;
          	public final BatteryProtectionDefinition batteryProtection;
          	public final String serialNrPrefixBms;
          	public final String serialNrPrefixModule;
          
          	private final int value;
          	private final String type;

Sn0w3y · January 19, 2025, 4:53pm

PR erstellt

TheSerapher · January 19, 2025, 4:55pm

Warst du schneller als ich Danke! Muss ich ein Auge auf das Changelog haben um das bei mir wieder in V zu ändern - hab es im Moment beim import manuell als mV angegeben.

benniju · January 21, 2025, 8:29am

In dem Commit wurde die einfach Einheit auf auf Millivolt geändert, der Wert bleibt aber eigentlich gleich, so wie ich das sehe

TheSerapher · January 21, 2025, 8:49am

Der Wert ist auch in Millivolt - nur die Einheit hatte nicht gestimmt.

Sn0w3y · January 21, 2025, 9:55am

Danke… genau so ist es - ist übrigends gemerged - soltle also bald auf die FEMSen gerollt werden