Skip to content
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

Airplay 2 Devices showing in Homekit #92

Closed
adavison007 opened this issue Jul 25, 2019 · 9 comments
Closed

Airplay 2 Devices showing in Homekit #92

adavison007 opened this issue Jul 25, 2019 · 9 comments
Labels

Comments

@adavison007
Copy link

So even with Do not show Airplay 2 devices checked, they appear in homekit? I have redone this several times and even deleted the accessory file, airplay 2 now appearing since update, didn't prior?

@ebaauw
Copy link
Owner

ebaauw commented Jul 25, 2019

What's you config.json? Could you please list the output of ph topology -v?

My zone players are too old for Airplay, but I understand you need to enable it, probably in the Sonos app settings. I needs to be enabled when homebridge-zp first runs (after deleting ~/.homebridge/accessories/cachedAccessories) and creates the accessories. See #88 (comment).

@adavison007
Copy link
Author

@ebaauw Here is the current JSON, Also I did have exclude checked prior to the first restart. I have also cleared the Accessory cache and they still remain... again this did work fine prior to the latest update I did yesterday.

I tried running ph topology -v in the command line and got nothing?

    {
        "name": "Sonos",
        "alarms": true,
        "excludeAirPlay": true,
        "nameScheme": "% Speakers",
        "service": "switch",
        "platform": "ZP"
    }

@ebaauw
Copy link
Owner

ebaauw commented Jul 26, 2019

It must be the heat, I meant zpinfo -H xx.xx.xx.xx topology -v, where the xx’s are the ip address of one of your zone players.

Homebridge-zp lists the zones (rooms) and zone players it has discovered during startup, incl. the detected capabilities per zone player. Like whether it supports Airplay. Is that reported correctly?

again this did work fine prior to the latest update I did yesterday.

Odd, I’ll double check the code what changes I made.

@adavison007
Copy link
Author

adavison007 commented Jul 26, 2019

zpinfo -H xx.xx.xx.xx topology -v

DEN, LILYS ROOM, ELLAS ROOM and KITCHEN are all Airplay

sonos.pdf

@ebaauw
Copy link
Owner

ebaauw commented Jul 26, 2019

Thanks. So they're listen correctly, aren't they?

When you say it worked before the upgrade: was that a version <1.0.0? It looks like the check for airPlay didn't make it through to the 1.0.0. code refactoring. I'll add it.

Unrelated: the zone name for the zone with the Sub isn't yet handled correctly. Could run zpinfo -H 192.168.1.225 eventlog. This will subscribe to the Kitchen Beam and log all events it sends. Just press ctrl-C to stop it. I'm particularly interested in the ZoneGroupTopology event.

@adavison007
Copy link
Author

Thanks. So they're listen correctly, aren't they?

When you say it worked before the upgrade: was that a version <1.0.0? It looks like the check for airPlay didn't make it through to the 1.0.0. code refactoring. I'll add it.

Unrelated: the zone name for the zone with the Sub isn't yet handled correctly. Could run zpinfo -H 192.168.1.225 eventlog. This will subscribe to the Kitchen Beam and log all events it sends. Just press ctrl-C to stop it. I'm particularly interested in the ZoneGroupTopology event.

The previous version being the latest one prior to the errors from yesterdays update,e not sure which version, sorry.

here's the log...
SUB.docx

@ebaauw
Copy link
Owner

ebaauw commented Jul 26, 2019

Thanks. So in a home theatre setup, the Sub is configured is a satellite, just like the surround speakers. What does the Sonos app show as room name for the Kitchen in the EQ Settings? Something like Kitchen (+SW+LS+RS)?

@adavison007
Copy link
Author

Thanks. So in a home theatre setup, the Sub is configured is a satellite, just like the surround speakers. What does the Sonos app show as room name for the Kitchen in the EQ Settings? Something like Kitchen (+SW+LS+RS)?

see attached

IMG_5495

ebaauw added a commit that referenced this issue Jul 26, 2019
Honour `excludeAirPlay` setting, see #92.
@ebaauw ebaauw added the bug label Jul 27, 2019
@ebaauw
Copy link
Owner

ebaauw commented Jul 27, 2019

Seems like I lost the check for Airplay devices in v1.0.0 and later. Will put it back in next version.

EDIT In v1.0.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants