Skip to content

Update zha.markdown #15829

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 10, 2020
Merged

Update zha.markdown #15829

merged 1 commit into from
Dec 10, 2020

Conversation

Jpsy
Copy link
Contributor

@Jpsy Jpsy commented Dec 5, 2020 •
edited
Loading

Proposed change

  • HA restart required to detect new modules (i.e. USB)
  • Add details on where to find serial device paths.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue:

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@probot-home-assistant probot-home-assistant bot added the current This PR goes into the current branch label Dec 5, 2020
frenck
frenck reviewed Dec 5, 2020
View reviewed changes
source/_integrations/zha.markdown
@@ -84,6 +84,8 @@ Some other Zigbee coordinator hardware may not support a firmware that is capabl

## Configuration - GUI

Connect your radio module and restart Home Assistant.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That should not be needed?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restart should not be needed with Home Assistant OS, but if not using Home Assistant OS as the installation method...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For whatever reason, after plugging a new USB stick into Running Home Assistant OS installation for the very 1st time, it does show as a serial port in the Supervisor Hardware tab, however accessing the serial port from the HA Core instance fails. Exactly same steps work after the host reboot.

frenck
frenck reviewed Dec 5, 2020
View reviewed changes
@frenck frenck added the in-progress This PR/Issue is currently being worked on label Dec 5, 2020
@Jpsy
Copy link
Contributor Author

Jpsy commented Dec 6, 2020

I am sorry, but I do not see any bug according to the docs.

Press Submit and the integration will try to detect radio type automatically. If unsuccessful, you will get a new pop-up asking for a radio type. ...:

This is exactly what happened.
The only problem is that the radio was not recognized in the first place. This is likely due to the fact that I plugged the USB radio (Conbee II) in and did not restart HA. After a restart the automatic detection worked perfectly. Without detection the flow of dialogs was correct too.

@frenck
Copy link
Member

frenck commented Dec 8, 2020

This is likely due to the fact that I plugged the USB radio (Conbee II) in and did not restart HA.

The point is... a restart to detect a USB device is not correct behavior, it should have been picked up.

@Hedda
Copy link
Contributor

Hedda commented Dec 8, 2020 •
edited
Loading

+1 for this likely being a bug. Restart of Home Assistant should not be needed with Home Assistant OS as the installation method.

https://www.home-assistant.io/docs/installation/

@Hedda
Copy link
Contributor

Hedda commented Dec 8, 2020

You could try to install/update to latest pyusb (libusb) with the command

python3 -m pip install pyusb>=1.1.0

https://pypi.org/project/pyusb/

Hedda added a commit to Hedda/zigpy-deconz that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
fe8fd1b 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when using USB adapters.

Reference home-assistant/home-assistant.io#15829 and zigpy/zigpy-zigate#71
@Hedda Hedda mentioned this pull request Dec 8, 2020
Closed
Hedda added a commit to Hedda/zigpy-znp that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
9d16b76 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when trying to add USB adapters.

Reference zigpy/zigpy-zigate#71 and home-assistant/home-assistant.io#15829
@Hedda Hedda mentioned this pull request Dec 8, 2020
Closed
Hedda added a commit to Hedda/zigpy-cc that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
54fe291 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when trying to add USB adapters.

Reference zigpy/zigpy-zigate#71 and home-assistant/home-assistant.io#15829
@Hedda Hedda mentioned this pull request Dec 8, 2020
Closed
Hedda added a commit to Hedda/zigpy-xbee that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
641792c 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when trying to add USB adapters.

Reference zigpy/zigpy-zigate#71 and home-assistant/home-assistant.io#15829
@Hedda Hedda mentioned this pull request Dec 8, 2020
Closed
Hedda added a commit to Hedda/bellows that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
4c23a85 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when trying to add USB adapters.

Reference zigpy/zigpy-zigate#71 and home-assistant/home-assistant.io#15829
Hedda added a commit to Hedda/bellows that referenced this pull request Dec 8, 2020
@Hedda
Update setup.py adding "pyusb>=1.1.0" requirement
0d30e83 
Update setup.py adding "pyusb>=1.1.0" requirement as probably does not hurt have latest pyusb / libusb installed when trying to add USB adapters.

Reference zigpy/zigpy-zigate#71 and home-assistant/home-assistant.io#15829
@Hedda Hedda mentioned this pull request Dec 8, 2020
Closed
@Adminiuga Adminiuga mentioned this pull request Dec 9, 2020
Closed
@frenck frenck merged commit 9ae3e02 into home-assistant:current Dec 10, 2020
@probot-home-assistant probot-home-assistant bot removed the in-progress This PR/Issue is currently being worked on label Dec 10, 2020
@Jpsy Jpsy deleted the patch-4 branch December 10, 2020 09:50
@github-actions github-actions bot locked and limited conversation to collaborators Dec 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Adminiuga Adminiuga Adminiuga left review comments

@Hedda Hedda Hedda left review comments

@frenck frenck frenck approved these changes

Assignees

@Jpsy Jpsy

Labels
current This PR goes into the current branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Jpsy @frenck @Hedda @Adminiuga