Sync the fork #3
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
* Add boost support for TRVs * Updated pyhiveapi dependency to 0.2.20.1
* Adds save_image * Update test_image_processing.py * Update test * Tidy test * update image_processing with reviewer comments * Update test_image_processing.py * Ammend tests not passing * Patch convert * remove join * Use valid_config_save_file
* add unique_id to directv entities. * add addiitional debug and fix tests. * fix lint error. * rework unique_id and flow a bit. * review adjustments. * review adjustments * review adjustments * review adjustments. * review adjustments * review adjustments * review adjustments * review adjustments * lint * use serial number for host unit and mac address for client units * fix elsif * update test with realistic client id * lint
* Add and use time related constants * Sort time constants and reuse them in data rate constants * Fix greeneyemonitor import * Fix greeneyemonitor import V2 * Fix failing here_travel_time test * Add TIME_MONTHS and TIME_YEARS * Use TIME_MINUTES in opentherm_gw and isy994 * Add and use TIME_MILLISECONDS * Change inconsistent ones * Add TIME_MICROSECONDS and TIME_WEEKS * Use constants in apcupsd and travisci * Fix import error in upnp sensor.py * Fix isy994 sensor.py using TIME_YEARS * Fix dyson tests * Use TIME_SECONDS in more places * Use TIME_DAYS in google wifi
* Use f-strings in integrations starting with A * Use f-strings in tests for integrations starting with A * Fix pylint by renaming variable * Fix nested for loop in f-string for aprs device_tracker * Break long lines into multiple short lines * Break long lines into multiple short lines v2
* Deduplicate code in the august integration * Add additional tests for august (more coming) * Door state is now updated when a lock or unlock call returns as the state is contained in the response which avoids the confusing out of sync state * revert * document known issue with doorsense and lock getting out of sync (pre-existing) * Address review comments * Additional review comments
* Fix SimpliSafe reconnection issue upon websocket error * Rename listener cancelation properties * Remove watchdog
* hookid : isort fix * New platform: Ezviz * updated CODEOWNERS for ezviz * proper test requirements * resolved conflict * regenerated requirements * removed stale comments, only one call to add_entities, removed unnecessary attributes * setup is sync, not async. Removed stale comments * Compatible with pyezviz 0.1.4 * pyezviz 0.1.4 is now requiredf * Added PTZ + switch management * added services.yaml * proper requirement * PTZ working in async mode * Now updates the entity * Compatible with pyezviz 0.1.5.1 * Fixed switch ir service registering * now requires pyezviz 0.1.5.2 * now requires pyezviz 0.1.5.2 * Revert "regenerated requirements" This reverts commit 848b317. * Rollbacked to a simpler version * snake_case names everywhere, logging sanatizing, voluptuous proper check * pyezviz 0.1.5, reworked the PR so that it's intelligible * no need for services.yaml for now * proper voluptuous validation * Removed stale code, use proper conf variable, describe attributes * regenerated requirements * stale * removed status from attributes, since we use it for available we don't need it here then. * Fixed log message
* Port homekit_controller to aiohomekit * Remove succeed() test helper * Remove fail() test helper
* Add support for Supla gate with sensor * Fix Supla switch module description and state access * Add docs to methods of Supla gate * Add missing comma * Remove unused import * Sort imports of Supla cover * Add returning availability for every Supla device * Use direct access to dict * Remove deprecated property "hidden" * Remove unused constant * Revert using get function on dict
* Speed up validate_entity_id * Add some more invalid entity IDs * Adjust regular expression * Extend and sort test cases * Update regular expression, more cases, faster * Adjust tests, allow start with number, disallow double underscore Co-authored-by: Franck Nijhof <frenck@frenck.nl>
* Use f-strings in integrations starting with B * Use f-strings in integrations starting with C * Use f-strings in integrations starting with D * Use f-strings in integrations starting with E * Fix pylint errors * Fix pylint errors v2 * Fix tests * Fix tests v2
* Handle unhandled IQVIA data update exception * Cleanup * Ask for forgiveness, not permission * Use warning-level logs * Fix log messages
* Modernize SimpliSafe config flow * Fix tests
* Modernize RainMachine config flow * Update strings
…32151) * use get to get volume_step since it is optional * always set volume_step to default in options and data if its not included
* Add tests for legacy Script helper behavior
* Add Script helper if_running and run_mode options
- if_running controls what happens if Script run while previous run
has not completed. Can be:
- error: Raise an exception
- ignore: Return without doing anything (previous run continues as-is)
- parallel: Start run in new task
- restart: Stop previous run before starting new run
- run_mode controls when call to async_run will return. Can be:
- background: Returns immediately
- legacy: Implements previous behavior, which is to return when done,
or when suspended by delay or wait_template
- blocking: Returns when run has completed
- If neither is specified, default is run_mode=legacy (and if_running
is not used.) Otherwise, defaults are if_running=parallel and
run_mode=background. If run_mode is set to legacy then if_running must
be None.
- Caller may supply a logger which will be used throughout instead of
default module logger.
- Move Script running state into new helper classes, comprised of an
abstract base class and two concrete clases, one for legacy behavior
and one for new behavior.
- Remove some non-async methods, as well as call_from_config which has
only been used in tests.
- Adjust tests accordingly.
* Change per review
- Change run_mode default from background to blocking.
- Make sure change listener is called, even when there's an unexpected
exception.
- Make _ScriptRun.async_stop more graceful by using an asyncio.Event for
signaling instead of simply cancelling Task.
- Subclass _ScriptRun for background & blocking behavior.
Also:
- Fix timeouts in _ScriptRun by converting timedeltas to float seconds.
- General cleanup.
* Change per review 2
- Don't propagate exceptions if call from user has already returned
(i.e., for background runs or legacy runs that have suspended.)
- Allow user to specify if exceptions should be logged. They will still
be logged regardless if exception is not propagated.
- Rename _start_script_delay and _start_wait_template_delay for
clarity.
- Remove return value from Script.async_run.
- Fix missing await.
- Change call to self.is_running in Script.async_run to direct test of
self._runs.
* Change per review 3 and add tests
- Remove Script.set_logger().
- Enhance existing tests to check all run modes.
- Add tests for new features.
- Fix a few minor bugs found by tests.
* Add and use speed constants * Add and use meter based volume and area constants * Add and use more mass unit constants * Add and use concentration unit constants * Add and use watts per square meter constant * Use more time constants * Use more data constants
The documentation for google_travel_time was at odds with the implementation. The documentation stated a default scan time of 5 minutes, but the implementation was using Throttle which resulted in the sensor updating at a maximum rate of one API call every 5 minutes. This was especially at odds with a given example at the end of the documentation, which showed updating the sensor every 2 minutes during commute times. This change brings the implementation in line with the docs by adopting the `SCAN_INTERVAL` constant set to the stated default of 5 minutes and removing the Throttle.
* Allow string values on zwave.set_node_value This allows for: * Accessing longer value_ids. In some cases, value ids in z-wave nodes are very large (17 digits in my case). Passing them as int does not seem to work well (python probably truncates the number), but passing them as string works fine * Changing color values, which are represented as hex string reformat test * update services.yaml with string set_node_value
* Add edl21 component for SML-based smart meters * edl21: Remove unused variable * [edl21] Add 1 minute throttle to the sensor * Update homeassistant/components/edl21/manifest.json Fix documentation URL Co-Authored-By: Paulus Schoutsen <paulus@home-assistant.io> * edl21: Move imports to top * edl21: Remove special case for STATE_UNKNOWN, which replicated default behavior * edl21: Implement blacklist for and warn about unhandled OBIS values * edl21: Make blacklist global * edl21: Add filter to issues URL Co-Authored-By: Paulus Schoutsen <paulus@home-assistant.io> * edl21: Rename device to entity * edl21: Don't schedule async_add_entities * edl21: Use dispatcher, implement own throttling mechanism * edl21: Simplify keeping track of known obis * edl21: Use whitelist for state attributes * edl21: Remove dispatcher on shutdown * edl21: Convert state attributes to snakecase * edl21: Annotate handle_telegram with @callback * edl21: Call async_write_ha_state instead of schedule_update_ha_state Co-authored-by: David Straub <straub@protonmail.com> Co-authored-by: Paulus Schoutsen <paulus@home-assistant.io>
* [soundtouch] workaround for API bug when removing multiple slaves from a zone at once * [soundtouch] added additional attributes exposing multiroom zone info * Fix update with slave entities * Add zone attributes test * Fix and clean up tests * Fix typo Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
this propose makes some logic to the device triggers, matching the events for the aqara cube. this also fixes the double tap function on side 2
* Emoncms API provides a Unit The EmonCMS API has been amended to include a 'unit' as part of it's payload. By using this information, all the sensors can be created without the need for individual sensors to be setup by type. The change is backward compatible so if a unit type has been specified in the configuration, this will be used by default. If no unit is pecified either by the Home Assistant configuration, or the Emoncms API, then the default of W will be used as before. * Update sensor.py Check the 'unit' key is in the API call. Older systems may not have that key in the payload. * Modified approach with new configuration item * Removed new config item Removed the configuration item. The integration attempts to get the unit from the API. If this fails *or* the unit key of the API is blank, either the specified unit, or the default will be used. If approved, documentation will be updated. * Update homeassistant/components/emoncms/sensor.py Co-Authored-By: springstan <46536646+springstan@users.noreply.github.com> * Update homeassistant/components/emoncms/sensor.py Co-Authored-By: springstan <46536646+springstan@users.noreply.github.com> * Apply suggestions from code review * Apply suggestions from code review v2 * Update homeassistant/components/emoncms/sensor.py Co-Authored-By: springstan <46536646+springstan@users.noreply.github.com> * Update sensor.py Update `config_unit` Co-authored-by: springstan <46536646+springstan@users.noreply.github.com>
* add async_setup_entry for remote platform * add async_unload_entry for remote platform * Update __init__.py * Update __init__.py * Update __init__.py * Update __init__.py * Update __init__.py * Update __init__.py * Update __init__.py * Type Co-authored-by: Paulus Schoutsen <paulus@home-assistant.io>
* Add attribute latest release tag * add attrs = and only create attr Tag if exists made requested changes * change condition for _latest_release_tag to use self._github_data.latest_release_url * Correct changes * Update sensor.py * blackify
* Modernize WWLLN config flow * Code review * Update tests
* Add block network access control to config option * Clean up
* added presets for switch devices * added channel type to __init and const * ran pylint on library so needed a few changes in names * removed callback * bool -> cv.boolean
…an update (#32400) * Fetch iCloud devices again if the status is pending * Remove "No iCloud device found" double check * fix default api_devices value * Remove useless unitialisation declarations
* Initial support for HomeKit enabled televisions * Fix nit from review
* Add Huawei LTE operator and network related sensors Adds "Operator search mode", "Operator name", "Operator code", and "Preferred mode" sensors * Blackify * Blackify "Add Huawei LTE operator and network related sensors"
* Require title, separate show sidebar option * Fix list command not updating * Some more test checks
* Allow teaching logbook about events * Use async_add_executor_job * Fix tests
* bugfix: correctly handle non-UTC TZs * bugfix: system mode is always permanent * bugfix: handle where until is none * tweak: improve logging to support above fixes
* Returns an unencrypted location of all persons with device trackers * Handle encrypted messages and exclude the poster's location * Friends is by default False. Reformats with Black * Updates the context init to account for the Friends option * Fix Linter error * Remove as a config option * No longer imports encyrption-related functions in encrypt_message * Fix initialization in test * Test the friends functionality * Bugfix for persons not having a location * Better way to return the timestamp * Update homeassistant/components/owntracks/__init__.py Co-Authored-By: Paulus Schoutsen <paulus@home-assistant.io> * Linting and tid generation * Fix test Co-authored-by: Paulus Schoutsen <paulus@home-assistant.io>
* add app support * code cleanup, add additional test, add CONF_APPS storage logic for import * simplify schema defaults logic * remove unnecessary lower() and fix docstring * remove default return for popping CONF_APPS during import update because we know entry data has CONF_APPS due to if statement * further simplification * even more simplification * fix type hints * move app configuration to separate step, fix tests, and only make app updates if device_type == tv * remove errors variable from tv_apps and move tv_apps schema out of ConfigFlow for consistency * slight refactor * remove unused error from strings.json * set unique id as early as possible * correct which dictionary to use to set unique id in pair_tv step
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.
No description provided.