Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Some hybrid inverters start with SD not SA #6

Closed
wants to merge 2 commits into from
Closed

Some hybrid inverters start with SD not SA #6

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0c92b35
Some hybrid inverters start with SD not SA
zaheerm Feb 6, 2022
29ed4a2
Merge branch 'main' into support_newer_serials
dewet22 Feb 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions givenergy_modbus/model/inverter.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,15 @@ def from_serial_number(cls, serial_number: str):
else:
raise UnknownModelError(prefix)

@classmethod
def from_serial_number(cls, serial_number: str):
models = {'CE': cls.AC, 'ED': cls.Gen2, 'SA': cls.Hybrid, 'SD': cls.Hybrid}
prefix = serial_number[:2]
if prefix in models:
return models[prefix]
else:
raise ModelUnknownError("Model unknown")


class Inverter(GivEnergyBaseModel):
"""Structured format for all inverter attributes."""
Expand Down