Skip to content

Profiles review improvements#4

Open
nikitug wants to merge 22 commits intomainfrom
profiles-review-improvements
Open

Profiles review improvements#4
nikitug wants to merge 22 commits intomainfrom
profiles-review-improvements

Conversation

@nikitug
Copy link
Member

@nikitug nikitug commented Feb 23, 2026

No description provided.

@nikitug nikitug requested review from anataty and kulti February 23, 2026 12:52
@nikitug nikitug force-pushed the profiles-review-improvements branch 3 times, most recently from bd4ea0e to a4ce28d Compare February 23, 2026 15:01
Copy link
Contributor

@kulti kulti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of questions to discuss before release.

@nikitug nikitug force-pushed the profiles-review-improvements branch 2 times, most recently from 303a2a4 to 1017e94 Compare February 24, 2026 08:06
Both battery_current and battery_power now use generator reference
frame: positive = discharging (energy out), negative = charging
(energy in). Previously current used the opposite convention, making
P = V * I inconsistent.

Also documents sign and naming conventions in README.
Add 'throttled' state between 'operating' and 'shutting_down' for
inverters operating at reduced power due to temperature derating,
grid requirements, or power limits. Common in SMA, Fronius, and
SolarEdge inverters.
@nikitug nikitug force-pushed the profiles-review-improvements branch 2 times, most recently from fb89170 to c6e7621 Compare February 24, 2026 09:52
Power factor is a dimensionless ratio; use UCUM unit "1" for consistency
with all other numeric telemetry attributes that specify a unit.
@nikitug nikitug force-pushed the profiles-review-improvements branch from fed2726 to 877549c Compare February 24, 2026 10:08
Sensor profile descriptions now describe what the device IS rather than
saying "A profile for...".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants