-
-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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 support for logger info in fronius integration #54795
Conversation
This adds the option to track the costs and returns of energy of the fronius inverter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When bumping a requirement, add a link to the release and of diff for that requirement to the PR description 👍
Sorry :) Diff/Release page added |
b2e6e05
to
76e90a5
Compare
76e90a5
to
ad808b8
Compare
3fdc9a5
to
eba4922
Compare
Co-authored-by: Franck Nijhof <frenck@frenck.nl>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @nielstron 👍
* upstream/dev: (52 commits) Add zigbee connection for wemo bridge lights (home-assistant#53813) Use EntityDescription - ring (home-assistant#55023) Add support for bridge accessory unavailability (home-assistant#52207) Fix P1 Monitor requirement in manifest (home-assistant#55027) Improve device trigger type hinting (home-assistant#54907) Use EntityDescription - openuv (home-assistant#55022) Add silver quality scale to goalzero (home-assistant#53299) Add Binary Sensor for WeMo Insight & Maker (home-assistant#55000) Add state_class to relevant sensors in Danfoss Air (home-assistant#54847) Remove base_test from modbus test harness (home-assistant#55018) Add support for logger info in fronius integration (home-assistant#54795) Skip trying the pyudev observer when using standalone docker for usb (home-assistant#54987) Upgrade watchdog to 2.1.4 (home-assistant#54993) Upgrade apprise to 0.9.4 (home-assistant#55002) Fix incorrect power device class on energy sensors in Smappee (home-assistant#54994) Add missing BYN currency (home-assistant#55001) Fix manual setup when roomba is on different subnet (home-assistant#54639) [ci skip] Translation update Bump python-yeelight to 0.7.3 (home-assistant#54982) Upgrade PyTurboJPEG to 1.5.2 (home-assistant#54992) ...
Breaking change
Proposed change
This adds the option to track the costs and returns of energy of the fronius inverter.
The newly added sensors "cash factor" and "delivery factor" track the current price for bought and sold energy in currency/kWh.
This is a feature of a newer version of pyfronius and hence requires a dependency upgrade.
New Release: https://github.com/nielstron/pyfronius/releases/tag/release-0.6.0
Diff: nielstron/pyfronius@release-0.5.5...release-0.6.0
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: