-
Notifications
You must be signed in to change notification settings - Fork 2
Description
pypvs library currently doesn't provide firmware and hardware version fields for inverters, meters, and ESS devices, causing a data regression compared to the legacy dl_cgi endpoint.
Old Firmware (dl_cgi endpoint) provided:
Inverters:
"hw_version": "4403",
"SWVER": "4.21.4"Battery devices:
"hw_version": "4.51",
"SWVER": "2.8"ESS device:
"hw_version": "0",
"SWVER": "0"Power Meters:
"SWVER": "3000"pypvs currently provides:
- ❌
PVSInvertermodel: No firmware/hardware fields - ❌
PVSESSmodel: No firmware/hardware fields - ❌
PVSMetermodel: No firmware/hardware fields - ✅
gateway(PVS device): Hassoftware_versionandhardware_version✓
Impact:
Home Assistant integrations using pypvs now show "Firmware: Unknown Hardware: Unknown" for all inverters, meters, and battery devices, which looks broken to users. This information was available in the old dl_cgi format and should be preserved in the new LocalAPI/pypvs implementation.
Requested Enhancement:
Add firmware/hardware version fields to pypvs models:
@dataclass(slots=True)
class PVSInverter:
serial_number: str
model: str
software_version: str # NEW
hardware_version: str # NEW
# ... existing fieldsSimilar additions needed for PVSESS and PVSMeter models.
Varserver Data Source:
The /sys/devices/inverter/, /sys/devices/ess/, and /sys/devices/meter/ varserver endpoints likely contain these fields (similar to how /sys/info provides gateway versions). The from_varserver() methods would need to extract them.
Workaround:
Currently using placeholders ("SWVER": "pypvs", "HWVER": model) to avoid showing "Unknown" in UI, but this loses valuable diagnostic information that was previously available.