Enphase envoy individual inverter production #24445
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
homeassistant
added
integration: enphase_envoy
small-pr
2 tasks
andrewsayre
previously requested changes
Jun 10, 2019
OttoWinter
reviewed
Jun 14, 2019
|
This pull request has been mentioned on Home Assistant Community. There might be relevant details there: https://community.home-assistant.io/t/trouble-with-enphase-envoy/67359/37 |
MartinHjelmare
requested changes
Jun 22, 2019
|
@MartinHjelmare Here's my first crack at breaking those attributes out into their own sensors. |
|
There are some merge conflicts. |
|
Derp, fixed conflicts. @MartinHjelmare @andrewsayre |
|
@andrewsayre I refactored enough that I don't think any of your requested changes still apply. |
|
@MartinHjelmare Seems like andrewsayre has ghosted me, can you merge this without his approval? |
I was not aware you were waiting on anything from me. Martin already approved the PR. :) |
KJonline
pushed a commit
to Rendili/home-assistant
that referenced
this pull request
Jul 8, 2019
… into hive_water_heater * 'hive_water_heater' of github.com:Rendili/home-assistant: (21 commits) Sensibo, add HVAC_MODE_OFF (home-assistant#25016) Add support for arcam fmj receivers (home-assistant#24621) Enphase envoy individual inverter production (home-assistant#24445) Implement Twilio SMS notify MediaUrl support (home-assistant#24971) Climate 1.0 (home-assistant#23899) Correct socket use in cert_expiry platform (home-assistant#25011) Added missing yeelight models mapping (home-assistant#24963) Install requirements for integrations in packages before importing them. (home-assistant#25005) Upgrade insteonplm to 0.16.0 and add INSTEON scene triggering (home-assistant#24765) Upgrade hdate==0.8.8 (home-assistant#25008) upgrade switchmate to latest lib (home-assistant#25006) Test dependency updates (home-assistant#25004) Add support for aurora ABB Powerone solar photovoltaic inverter (home-assistant#24809) Sleepiq single sleeper crash (home-assistant#24941) Changes as per code review of home-assistant#24646 (home-assistant#24917) Upgrade mypy to 0.711, drop no longer needed workarounds (home-assistant#24998) Adds Stale Probot for issues (home-assistant#24985) Adds Lock Threads Probot (home-assistant#24984) Switched from tuyapy to tuyaha as 1st one is not maintained (home-assistant#24821) Fix errors if rest source becomes unavailable (home-assistant#24986) ... # Conflicts: # homeassistant/components/hive/__init__.py # homeassistant/components/hive/climate.py
Merged
alandtse
pushed a commit
to alandtse/home-assistant
that referenced
this pull request
Oct 12, 2019
* bump envoy_reader version to 0.4 * bump dependency envoy_reader to 0.4 * Enphase envoy get individual inverter production * Add period in function description * Fix dumb typo * Define _attributes in __init__ * Better error messages, make update async * Fix format error * Fix pylint errors * set unknown state to None * Bump envoy_reader version to 0.8 * Change attributes to separate sensors * Fix dumb thing * Improve platform_setup for inverters * Remove unneeded self._attributes, refactor platform setup * Refactor platform setup
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description: Enphase Envoy component, get production values from individual inverters
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#9600
Checklist:
tox. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
python3 -m script.hassfest.requirements_all.txtby runningpython3 -m script.gen_requirements_all..coveragerc.If the code does not interact with devices: