home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.3k stars 30.62k forks source link

Enhancement: Enphase Production Current #123763

Closed sumur1au closed 1 month ago

sumur1au commented 2 months ago

The problem

Is it possible to include sensors for production currents on all three phases from the Envoy Enphase system. This would be useful in setting charger currents for my electric car.

What version of Home Assistant Core has the issue?

core-2024.8.1

What was the last working version of Home Assistant Core?

core-2024.8.1

What type of installation are you running?

Home Assistant OS

Integration causing the issue

enphase_envoy

Link to integration documentation on our website

No response

Diagnostics information

Enhancement issue and not a bug

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 2 months ago

Hey there @bdraco, @cgarwood, @joostlek, @catsmanac, mind taking a look at this issue as it has been labeled with an integration (enphase_envoy) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `enphase_envoy` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign enphase_envoy` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


enphase_envoy documentation enphase_envoy source (message by IssueLinks)

catsmanac commented 2 months ago

Hi @sumur1au, think that is technically possible.

At this moment there is already the current_power_production in watts for the phases which is directly related to the currents and might be used with some calculation for this?

sumur1au commented 2 months ago

Thanks Arie, I am calculating it now via watts, however it can be inaccurate as it don’t have the instantaneous line voltages.

On Wed, 14 Aug 2024 at 7:36 PM, Arie Catsman @.***> wrote:

Hi @sumur1au https://github.com/sumur1au, think that is technically possible.

At this moment there is already the current_power_production in watts for the phases which is directly related to the currents and might be used with some calculation for this?

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/123763#issuecomment-2288511478, or unsubscribe https://github.com/notifications/unsubscribe-auth/APXG5UUN6IS7I6XW2QXSM6LZRM6MXAVCNFSM6AAAAABMNNB2BOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBYGUYTCNBXHA . You are receiving this because you were mentioned.Message ID: @.***>

catsmanac commented 2 months ago

Ok, think we can add amp, but it will take some time to go through the review process.

sumur1au commented 1 month ago

Thanks for implementing the features. However, I can't see any of the new Current entities for production, I have checked the disabled column. I am not sure if I am missing anything.

catsmanac commented 1 month ago

Coding is completed, but it's not in a release yet. Not sure when it will appear, but probably in 2024.10 as it's a new feature, not a bugfix. New features get mostly included in the monthly releases.

sumur1au commented 1 month ago

@catsmanac Thanks