RFE - Support for Multiroom feature of Chromecast Audio

[home-assistant-libs]

RFE - Support for Multiroom feature of Chromecast Audio

[am0s]

Hi,

I just got myself two audio devices so I will have a look and see what needs to be done to support multiroom.
As for seeing the device twice is half-way correct, the group should appear together with the individual devices, just like it does in Apps that supports them.
But there is some missing code in terms of getting the correct name and connecting to the group device.
The group device seems to use a different port than the normal ones so we need add support for that.

I think I should have something working soon.

[am0s]

Multi room support is now completed, check out release v0.7.
https://github.com/balloob/pychromecast/releases/tag/v0.7

[michaelwingender]

Works like a charm with python2.7. Thanks a lot!!!

There seems only to be an issue with python3. Is it related to the multiroom feature or is it another issue?

mickey@egginerjoch ~/pychromecast-master $ python3
Python 3.4.2 (default, Oct 19 2014, 13:31:11)
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.

from future import print_function
import time
import sys
import logging
import pychromecast
pychromecast.get_chromecasts_as_dict().keys()
Exception in thread Thread-3:
Traceback (most recent call last):
File "/usr/lib/python3.4/threading.py", line 920, in _bootstrap_inner
self.run()
File "/usr/local/lib/python3.4/dist-packages/zeroconf.py", line 1063, in run
handler(self.zc)
File "/usr/local/lib/python3.4/dist-packages/zeroconf.py", line 1014, in
state_change=state_change,
File "/usr/local/lib/python3.4/dist-packages/zeroconf.py", line 931, in fire
h(*_kwargs)
File "/usr/local/lib/python3.4/dist-packages/zeroconf.py", line 989, in on_change
listener.add_service(_args)
File "/home/mickey/pychromecast-master/pychromecast/discovery.py", line 49, in add_service
model_name = model_name.decode('utf-8')
AttributeError: 'NoneType' object has no attribute 'decode'

dict_keys([])

[am0s]

There's a fix in the master branch, seems to be doing the trick.
Could you try it out?

If there are no more problems I'll make a 0.7.1 release.

[michaelwingender]

It is working fine. Thanks again. :-)

[balloob]

I did a quick pass through the code, it looks all good now.