-
-
Notifications
You must be signed in to change notification settings - Fork 32k
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
Upgrade pychromecast to 1.0.3 #11410
Conversation
Does this also fix the google group (that is created in the home app) where the id will randomly change and HA is still holding onto the previous id? This would typically end in a failure and traceback and HA could not connect to the cast device. |
@edif30 I don't think so. Sadly, I currently don't have my Chromecast Audios setup in order to reproduce your bug. But, looking at the commits since the last version, I don't see anything related to that problem. On a side note: I haven't looked into it in detail, but I believe the problem you're describing is probably very hard, or even impossible to fix, since there's probably no way to distinguish between a cast device that is temporarily offline and a renamed audio group. |
HI, Just did the update and I am still experiencing issues. None of my cast devices are showing up. I have 3 LG H51 and an SH6 Soundbar. I am using Home assistant in Docker (with host) on my Synology NAS This was working before, but since all this change started none of my devices have reappeared. We use this to service to notify us of medication times (for our nanny to give to our son) so this is rather frustrating that all this change has caused my devices to stop working… The entries in my logs are as follows:
Please can someone help? |
@jjanderson I agree, that's definitely not good to happen. So, if I understand correctly, all of your devices fail to show up in Home Assistant (i.e. they don't even appear and aren't just unresponsive). This makes me believe the error is caused by the changes in pychromecast/dial.py and your devices apparently not supporting the new pip install -U pychromecast
python
> import pychromecast
> pychromecast.get_chromecasts()
> pychromecast.Chromecast('IP_OF_YOUR_CAST_DEVICE') For an improved debugging experience it might be good to add some error logging statements here and here and not just ignore the connection errors completely. If my theory is correct (which would be quite weird, since the Google Home app uses the |
Hi, Yes, none of my Chromecast devices (I even have an Android Phone with Cast receiver running which does not display) show up in Home Assistant, however I can see them on the network as well as discover them on Google Home Appication on my iPhone. Do I need to run this command on the NAS or another linux machine with Python? Sorry, I very rarely venture into the CLI... |
|
There is definately something dodgy going on with cast.. and I am not the only one... https://community.home-assistant.io/t/media-player-not-appearing/8957 I would recommend annotating the cast component until all these bugs have been ironed out.... This whole issue is also a relativly hot topic at the moment which may or may not be related.... http://www.bbc.co.uk/news/technology-42745214 Regardless of if this is related or not, until such time as we (as a community) are 100% that a component is working, we shoud update (with a caveat) the notes in the component page... if we know there are issues, these should be called out, otherwise new users will simply lose faith in the prodct as a whole... Be honest... This is a google issue, not yours... but instead of newbie's trying and failing to connect because of hardware, and then losing faith in HA, we should call it out that it is work in progress... |
@jjanderson Firstly, to my understanding, that Chromecast/Google Home WiFi flooding issue is not related to this (trying to avoid terms like certainly here, as software bugs can always have some complicated causes - though this being the cause would be very unlikely). I understand this bug needs to be fixed, and preferably quickly. I completely agree Home Assistant should support all devices without breaking anything - in fact, this version bump was specifically intended to do exactly that, fix Google Home devices that previously weren't showing up in Home Assistant. If many users are experiencing this problem, another roll back might sadly be the best option. Then, waiting for much more feedback and improving logging before trying another version bump would be good too. Now, to fix the issue: I've gone over the diff of versions 0.8.2 and 1.0.3 of pychromecast and the only major change was the one in pychromcast/dial.py - which would support my previous theory. But, on the other hand, in the forum post you linked to you mentioned you were using discovery for cast. In that case, I would expect all the devices to work, because the changed code is only ever really needed when manually setting up a device via IP. Thank you very much for running the commands - though they are sadly not as revealing as I hoped them to be. Well, could you maybe try to open this url in a browser and report the output? |
@jjanderson After lots of investigating, I believe this issue is not caused by my first theory, since, as you verified, the new endpoint works. Instead, I now think it is most likely caused by some complicated network interface stuff (of which I have very little knowledge, #11431 has an excellent description)... I don't know if anybody has suggested this as a work-around yet, but you should still be able to override discovery by manually adding all the cast devices like this: media_player:
- platform: cast
host: IP_OF_YOUR_CAST_FIRST_DEVICE
- platform: cast
host: IP_OF_YOUR_CAST_SECOND_DEVICE Hope this helps. |
Description:
This is basically redoing the #10728 PR, which bumped pychromecast to 1.0.2. In a nutshell, the upgrade of pychromecast to 1.0.2 was supposed to fix an issue causing cast devices with never firmwares not being added to Home Assistant. But that fix introduces a new, probably even worse bug, that caused all cast devices to basically stop working - that bug, however, seems to have been fixed with 1.0.3.
In order to prevent a second google cast platform outage for Home Assistant, this time I asked for some feedback in #9965 before creating a new PR - and as far as I've heard, 1.0.3 seems to work fine.
Related issue (if applicable): fixes #11192, #11117, #9965
Checklist:
If the code communicates with devices, web services, or third-party tools:
tox
run successfully. Your PR cannot be merged unless tests passREQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
.