Bring nobo_hub to Bronze quality scale #168638
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,91 @@ | ||
| rules: | ||
|
Member
There was a problem hiding this comment. It would be cool if we can look into more typing
Contributor
Author
There was a problem hiding this comment. In the works - waiting for final review on the upstream library: echoromeo/pynobo#43
Member
There was a problem hiding this comment. Why is the HVAC mode linked hard to the preset mode?
Contributor
Author
There was a problem hiding this comment. It's explained in the documentation: https://www.home-assistant.io/integrations/nobo_hub/ The zones (normally rooms) operate either on a weekly schedule (auto) or in override mode. 3 different overrides are supported: normal heating, eco heating, away (non-freeze). The overrides are implemented as presets.
Member
There was a problem hiding this comment. Why do we occasionaly update
Contributor
Author
There was a problem hiding this comment. The hub does not push preset changes due to weekly schedule, so the climate entity has to be polled once per minute to pick up when the preset changes due to time passing. All other data are pushed from the hub on a TCP socket.
Member
There was a problem hiding this comment. Yes but what does that do? The docstring mentions
Contributor
Author
There was a problem hiding this comment. Clarified docstring. Hope it's OK to fix in this PR :)
Contributor
Author
There was a problem hiding this comment. Do you have some context for whats overwritten?
Member
There was a problem hiding this comment. Global override, I am not sure what that does from the docs
Contributor
Author
There was a problem hiding this comment. It overrides current mode for all zones. We could add a link to the mobile app manual if it helps:
joostlek marked this conversation as resolved.
joostlek marked this conversation as resolved.
|
||
| # Bronze | ||
| action-setup: | ||
| status: exempt | ||
| comment: Integration does not register custom actions. | ||
| appropriate-polling: done | ||
| brands: done | ||
| common-modules: done | ||
| config-flow-test-coverage: | ||
| status: done | ||
| comment: > | ||
| Tests driven to terminal CREATE_ENTRY or ABORT in PR #170141. | ||
| config-flow: done | ||
| dependency-transparency: done | ||
| docs-actions: | ||
| status: exempt | ||
| comment: Integration does not register custom actions. | ||
| docs-high-level-description: done | ||
| docs-installation-instructions: done | ||
| docs-removal-instructions: done | ||
| entity-event-setup: done | ||
| entity-unique-id: done | ||
| has-entity-name: done | ||
| runtime-data: done | ||
| test-before-configure: done | ||
| test-before-setup: done | ||
| unique-config-entry: done | ||
|
|
||
| # Silver | ||
| action-exceptions: | ||
| status: todo | ||
| comment: > | ||
| Entity actions (climate set_hvac_mode/set_preset_mode/set_temperature, | ||
| select select_option) currently raise unwrapped exceptions; will wrap | ||
| in HomeAssistantError with translation keys. | ||
| config-entry-unloading: done | ||
| docs-configuration-parameters: done | ||
| docs-installation-parameters: done | ||
| entity-unavailable: todo | ||
| integration-owner: done | ||
| log-when-unavailable: todo | ||
| parallel-updates: done | ||
| reauthentication-flow: | ||
| status: exempt | ||
| comment: The hub does not require authentication. | ||
| test-coverage: | ||
| status: done | ||
| comment: > | ||
| Investigate whether the `_spec_hub` helper in `test_init.py` can be | ||
| replaced by the conftest base mock. | ||
|
|
||
| # Gold | ||
| devices: | ||
| status: done | ||
| comment: > | ||
| Model name "Nobø Ecohub" under review for rename to "Nobø Hub". | ||
| diagnostics: todo | ||
| discovery: done | ||
| discovery-update-info: todo | ||
| docs-data-update: todo | ||
| docs-examples: todo | ||
| docs-known-limitations: todo | ||
| docs-supported-devices: todo | ||
| docs-supported-functions: todo | ||
| docs-troubleshooting: todo | ||
| docs-use-cases: todo | ||
| dynamic-devices: todo | ||
| entity-category: todo | ||
| entity-device-class: | ||
| status: todo | ||
| comment: > | ||
| Custom device class on global override select being dropped in | ||
| PR #170135. | ||
| entity-disabled-by-default: todo | ||
| entity-translations: todo | ||
| exception-translations: todo | ||
| icon-translations: todo | ||
| reconfiguration-flow: todo | ||
| repair-issues: | ||
| status: exempt | ||
| comment: Integration has no repair scenarios. | ||
| stale-devices: todo | ||
|
|
||
| # Platinum | ||
| async-dependency: done | ||
| inject-websession: | ||
| status: exempt | ||
| comment: Integration uses a local TCP socket (via pynobo); no HTTP client is used. | ||
| strict-typing: | ||
| status: todo | ||
| comment: Requires release of pynobo 1.9.0 | ||
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.
_attr_target_temperature_step = 1could useSTEP_WHOLE, just a nitThere 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.
Did you mean
PRECISION_WHOLEor as a local constant?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.
yes, precision
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.
Done in this PR