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

Add computed voltage and current to aggregates from TEDAPI status data #107

Merged
merged 2 commits into from
Jul 21, 2024

Conversation

jasonacox
Copy link
Owner

v0.10.9- TEDAPI Voltage & Current

  • Add computed voltage and current to /api/meters/aggregates using system data from TEDAPI status payload.

@jasonacox
Copy link
Owner Author

Proxy test: jasonacox/pypowerwall:0.10.9t63-beta

@jasonacox jasonacox merged commit 87ae55a into main Jul 21, 2024
25 checks passed
@jasonacox jasonacox deleted the v0.10.9 branch July 21, 2024 23:15
@BuongiornoTexas
Copy link

@jasonacox - I think I've run into a wrinkle with this - at least assuming this is used for system vitals. I've only just upgraded to 4.4.6 (been busy) and while I can see the system vitals, the voltages being reported for solar, home and grid are around the 415V level, while the powerwall reports around 235V. I'm guessing the 415V is being calculated assuming 120V/phase? I'd expect all of the voltages to be around 230-240V locally.

I did look around both here and in Powerwall-Dashboard, but didn't see any issues addressing this. If there is a live discussion, could you point me at it. If not, would you prefer an issue raised here or under Powerwall-Dashboard?

@jasonacox
Copy link
Owner Author

Yes, let's open an issue. Where do you see the bad voltages? When you get a chance, send me the raw data you see for /api/meters/aggregates (e.g. http://localhost:8675/api/meters/aggregates)

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.

2 participants