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

Master #3

Merged
merged 343 commits into from Jun 29, 2019

Conversation

@jcoin
Copy link
Owner

commented Jun 29, 2019

Breaking Change:

Description:

Related issue (if applicable): fixes #

Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>

Example entry for configuration.yaml (if applicable):

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
  • Untested files have been added to .coveragerc.

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.
pvizeli and others added 30 commits Jun 5, 2019
Bump simplipy to 3.4.2 (home-assistant#24326)
* Bump simplipy to 3.4.2

* Updated requirements
Meteofrance fix 24244 (home-assistant#24315)
* Update meteofrance package version

Fix home-assistant#24244

* Add code owner to manifest

* Update CODEOWNERS
Only update media icon when necessary (home-assistant#24324)
* Only update media icon when necessary

* Lint

* Comment
Add exception handling for Netatmo climate (home-assistant#24311)
* Add exception handling

* Make pylint happy
Use node descriptor from Zigpy for ZHA (home-assistant#24316)
* use zigpy node descriptor

* cleanup
Fix set_cover_position of the xiaomi_aqara cover for LAN protocol v2 (h…
…ome-assistant#24333)

* Fix set_cover_position of the xiaomi_aqara cover for LAN protocol v2 (Closes: home-assistant#24293)

* Fix lint
Add Environment Canada weather, sensor, and camera platforms (home-as…
…sistant#21110)

* Added Environment Canada weather platform

* Added Environment Canada weather platform

* Migrate to new folder structure

* Migrate to new folder structure

* Fix updates

* Fix updates again

* Bump env_canada to 0.0.4

* Bump env_canada to 0.0.4

* Bump env_canada to 0.0.4 in requirements_all.txt

* Change daily forecast timestamp and high/low test

* Change daily forecast timestamp and high/low test

* Bump env_canada to 0.0.5

* Break alerts into multiple sensors, bump env_canada to 0.0.6

* Bump env_canada to 0.0.7

* Remove blank line

* Remove 'ec' sensor prefix, bump env_canada to 0.0.8

* Corrections

* Change to manifests.json

* Add docstring to __init.py__

* Update CODEOWNERS

* pylint correction

* pylint correction

* Add alert details, bump env_canada to 0.0.9

* Update requirements_all.txt

* Update .coveragerc

* Bump env_canada to 0.0.10

* Update requirements_all.txt
Google Cloud Platform component (TTS) (home-assistant#23629)
* Added Google Cloud TTS service component feature

* Added Neutral voice gender

* Added line break at the end of files

* Updated CODEOWNERS, reqirements_all.txt and .coveragerc

* Fixed some ci/circleci: static-check errors

* Fixed some ci/circleci: static-check error

* Fixed some ci/circleci: pylint errors

* Fixed some ci/circleci: pylint errors

* * made supported_options const
* fixed direct env variable access

* Fixed import order

* * Component renamed
* Added encoding parameter
* Other fixes

* Changed folder name in .coveragerc

* * Removed whitespaces in blank lines
* Split long line

* Removed whitespaces in blank lines

* ci/circleci: static-check

* Fixed requirements_all.txt

* Added speed, pitch and gain parameters

* Added speed, pitch and gain as supported options

* Split too long line

* * Added profiles parameter
* Changed supported languages and encodings values
* Added parameters validations

* Fixes

* Fixes

* Fixes

* Fixes

* Fixes

* Changed options validation

* Added ToggleEntity save and restore state mechanism

* Revert "Added ToggleEntity save and restore state mechanism"

This reverts commit 0e27501
Add Streamlabs Water Monitor (home-assistant#21205)
* Add Streamlabs Water Monitor

* Fail Streamlabswater component setup when given invalid parameters

The Streamlabs Water component is unable to recover if it is given
an invalid API key or location id so this change is to ensure
we validate they are correct during setup and return a failure
if they are not.

* Prime Streamlabswater component sensors so data is available immediately

The sensors for the component were not causing an immediate load of
data from the API when being set up so there was some lag after
startup before values would show up.  This change does an explicit
update when the sensors are setup to ensure data is viewable
immediately after startup.

* Switch Streamlabswater logging to use %s for string formatting

* Update Streamlabswater component with correct dependencies

Dependencies were incorrectly specified using DEPENDS rather
than DEPENDENCIES

* Streamlabswater pull request feedback

Remove detailed class docstrings since they're in the documentation,
reduce code duplication in sensor classes, and remove periods from
the end of log messages.

* Reduce line length in Streamlabswater sensor

* Add docstring on Streamlabswater service callback method

* Get rid of unnecessary initializers in Streamlabswater sensor

* Add manifest file for Streamlabs Water Monitor

* Remove unused REQUIREMENTS
Bring the Sisyphus integration to silver quality (home-assistant#22457)
* Bring the Sisyphus integration to silver quality

Checklist:
- [x] (N/A - push integration) Set an appropriate SCAN_INTERVAL (if a polling integration)
- [x] Raise PlatformNotReady if unable to connect during platform setup
- [x] (N/A - no credentials) Handles expiration of auth credentials. Refresh if possible or print correct error and fail setup. If based on a config entry, should trigger a new config entry flow to re-authorize.
- [x] (N/A - local integration) Handles internet unavailable. Log a warning once when unavailable, log once when reconnected.
- [x] Handles device/service unavailable. Log a warning once when unavailable, log once when reconnected.
- [x] Set available property to False if appropriate
- [x] Entities have unique ID (if available)

* Feedback (fix a couple verbose places)

* Use a task instead of a lock

* Initialize field in constructor

* Revert package upgrade.
Support for Salda Smarty XV/XP Ventilation Unit (home-assistant#21491)
* Support for Salda Smarty XV/XP Ventilation Unit

* Update binary_sensor.py

* Update fan.py

* Update sensor.py

* Update __init__.py
Upgrade ruamel.yaml to 0.15.97 (home-assistant#24350)
* Upgrade ruamel.yaml to 0.15.97

* Fix req
Refactor Waze Travel Time & Update Requirements (home-assistant#22428)
* Refactor Waze Travel Time & Update Requirements

Refactored Waze Travel Time to contain a data object.

Changed error retrieving data to a warning.
Added distance conversion depending on region.
Removed dependency on TRACKABLE_DOMAINS list.
Update to use WazeRouteCalculator 0.10

3rd time's a charm.  Deleted fork, caused last PR to screw up.  So here we are.

* Update requirements_all.txt

* Revert package upgrade.

* Revert package upgrade.
Add service to update core location (home-assistant#24328)
* Add service to update core location

* Update test_init.py
cgtobi and others added 6 commits Jun 25, 2019
Update Waze route dependency to 0.10 (home-assistant#24754)
* Update manifest.json

Update waze calculator to 0.10, this was supposed to have been done in home-assistant#22428 but was missed. See discussion [here](https://community.home-assistant.io/t/waze-travel-time-update/50955/201)

* Update requirements_all.txt
Ignore duplicate tradfri discovery (home-assistant#24759)
* Ignore duplicate tradfri discovery

* Update name
@pascalsaul

This comment has been minimized.

Copy link

commented on d041c62 Jun 26, 2019

Oh that's why I can't close and open my HorizontalAwning anymore by the arrows.

I can only open and close it by changing the position to 0 or 100 now since 0.95 and not with the arrow up and down.

So why is this changed?

Debug information: cover.awning_back | closed | current_position: 0 (=cover closed)

This comment has been minimized.

Copy link
Member

replied Jun 27, 2019

@pascalsaul Please do not comment on closed merge requests or commits. That is not trackable and productive.
Create an issue on the GitHub repository instead.

Thanks already!

pnbruckner and others added 20 commits Jun 26, 2019
Fix netatmo weatherstation setup error (home-assistant#24788)
* Check if station data exists and reduce calls

* Fix module names list

* Add warning

* Remove dead code
Bump ZHA quirks module (home-assistant#24802)
* bump quirks version

* bump version - mija magnet
Alexa sync state report (home-assistant#24835)
* Do a sync after changing state reporting

* Fix entity config being None
Improve autodiscovered yeelights model detection (home-assistant#24671)
* Improve autodiscovered yeelights model detection

* Lint fixes

* Logger warn fix

@jcoin jcoin merged commit f8ddc74 into jcoin:master Jun 29, 2019

@jgracio

This comment has been minimized.

Copy link

commented on 20ba80f Jul 18, 2019

Hi
I don't know if I'm dropping this comment on the wrong place but I couldn't find another one.

I have been working with asuswrt on Home Assistant and now it is stopping several times per day disbling the linked automation.

Can you please help me or let me know who should I contact to learn how to get this fixed?

Thanks

The error I get is on HA log file is:
`2019-07-17 21:08:05 ERROR (MainThread) [aioasuswrt.connection] Host timeout.
2019-07-17 21:08:05 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/local/lib/python3.7/site-packages/aioasuswrt/connection.py", line 38, in async_run_command
"%s && %s" % (_PATH_EXPORT_COMMAND, command)), 9)
File "/usr/local/lib/python3.7/asyncio/tasks.py", line 423, in wait_for
raise futures.TimeoutError()
concurrent.futures._base.TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/device_tracker/setup.py", line 150, in async_device_tracker_scan
found_devices = await scanner.async_scan_devices()
File "/usr/src/homeassistant/homeassistant/components/asuswrt/device_tracker.py", line 36, in async_scan_devices
await self.async_update_info()
File "/usr/src/homeassistant/homeassistant/components/asuswrt/device_tracker.py", line 52, in async_update_info
self.last_results = await self.connection.async_get_connected_devices()
File "/usr/local/lib/python3.7/site-packages/aioasuswrt/asuswrt.py", line 166, in async_get_connected_devices
dev = await self.async_get_wl()
File "/usr/local/lib/python3.7/site-packages/aioasuswrt/asuswrt.py", line 102, in async_get_wl
lines = await self.connection.async_run_command(_WL_CMD)
File "/usr/local/lib/python3.7/site-packages/aioasuswrt/connection.py", line 48, in async_run_command
del self._client
AttributeError: _client`

This comment has been minimized.

Copy link
Member

replied Jul 18, 2019

Hi there @jgracio! Please do not comment on closed PR's or commits. Instead, open up an issue on GitHub in case you suspect an issue or go to our forums or hop on to Discord chat for support.

Thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
You can’t perform that action at this time.