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

Iris iL07_1 Missing Temperature/Humidity #4552

Closed
kbickar opened this issue Oct 4, 2020 · 10 comments · Fixed by Koenkk/zigbee-herdsman-converters#1630
Closed

Iris iL07_1 Missing Temperature/Humidity #4552

kbickar opened this issue Oct 4, 2020 · 10 comments · Fixed by Koenkk/zigbee-herdsman-converters#1630
Labels
problem Something isn't working

Comments

@kbickar
Copy link

kbickar commented Oct 4, 2020

What happened

The Iris iL07_1 device has a temperature/humidity sensor and reports those values, but they are not published by zigbee2mqtt

What did you expect to happen

temperature/humidity values should be visible

How to reproduce it (minimal and precise)

Pair device, view fields published

Debug info

Zigbee2MQTT version: 1.15.0
Adapter hardware: CC2531
Adapter firmware version: sonoff version

@TilmanK
Copy link
Contributor

TilmanK commented Oct 5, 2020

I've submitted a PR, once Koen merges it and updates the dev branch you'll need to test if it works for you.

@Koenkk
Copy link
Owner

Koenkk commented Oct 5, 2020

@kbickar can you test the changes done by @TilmanK, might be that we also need to setup reproting/binding (like https://github.com/Koenkk/zigbee-herdsman-converters/blob/a8e5c760f0b1d3e8a0d8445c94e8d2b2b5da36fd/devices.js#L3383). Thanks for helping @TilmanK

See https://www.zigbee2mqtt.io/how_tos/how_to_support_new_devices.html on how to modify devices.js

@kbickar
Copy link
Author

kbickar commented Oct 5, 2020

It's not actually my device, but I got my friend to test it out and after adding to the homeassistant.js config:
'iL07_1': [cfg.binary_sensor_occupancy, cfg.binary_sensor_tamper, cfg.binary_sensor_battery_low, cfg.sensor_temperature, cfg.sensor_humidity],

The sensors are showing up with "unknown" values so it sounds like it might need something else

@TilmanK
Copy link
Contributor

TilmanK commented Oct 7, 2020

It's not actually my device, but I got my friend to test it out and after adding to the homeassistant.js config:
'iL07_1': [cfg.binary_sensor_occupancy, cfg.binary_sensor_tamper, cfg.binary_sensor_battery_low, cfg.sensor_temperature, cfg.sensor_humidity],

The sensors are showing up with "unknown" values so it sounds like it might need something else

Can you confirm you tried with my suggested changes?

@kbickar
Copy link
Author

kbickar commented Oct 7, 2020

Yes, that was with the updated devices.js

@Koenkk
Copy link
Owner

Koenkk commented Oct 16, 2020

So to confirm, with these changes: https://github.com/Koenkk/zigbee-herdsman-converters/pull/1630/files you get temperature and humidity values for this device?

@kbickar
Copy link
Author

kbickar commented Oct 16, 2020

No values were reported

@Koenkk
Copy link
Owner

Koenkk commented Oct 16, 2020

Can you try with:

{
    zigbeeModel: ['1117-S'],
    model: 'iL07_1',
    vendor: 'Iris',
    description: 'Motion Sensor',
    supports: 'motion, tamper, temperature, humidity and battery',
    fromZigbee: [fz.ias_occupancy_alarm_2, fz.temperature, fz.humidity],
    toZigbee: [],
    exposes: [exposes.boolean('occupancy'), exposes.boolean('battery_low'), exposes.boolean('tamper')],
    meta: {configureKey: 1},
    configure: async (device, coordinatorEndpoint) => {
        const endpoint = device.getEndpoint(1);
        await bind(endpoint, coordinatorEndpoint, ['msTemperatureMeasurement', 'msRelativeHumidity']);
        await configureReporting.temperature(endpoint);
        await configureReporting.humidity(endpoint);
    },
},

After this manually configure via: https://www.zigbee2mqtt.io/information/mqtt_topics_and_message_structure.html#zigbee2mqttbridgeconfigure (make sure to wakeup the device right before executing it by clicking a button on it) and check the log if OK?

@kbickar
Copy link
Author

kbickar commented Nov 11, 2020

I commented on the PR the config that was working.

I added the line

exposes: [e.occupancy(), e.battery_low(), e.temperature(), e.humidity()],

After that was added it worked and the HA discovery messages were correct.

I also checked and the device physically has no tamper sensor so that can be removed

@Tediore
Copy link

Tediore commented Feb 20, 2021

Running 1.17.1. I have three of these motion sensors and none of them would report temp or humidity without adding what Koenkk mentioned above to my devices.js (#4552 (comment))

Scratch that, it looks like reporting for temperature and humidity just needs to be set up for it to work. I just pulled a fresh container to undo my changes to devices.js and paired another one of these sensors. The other three are still reporting temp/humidity (because reporting is set up), but this new one isn't. I'll check for temp/humidity messages from this new sensor in a few hours and update my message (I suspect there won't be any).

Edit: Can confirm reporting needs to be enabled for temperature/humidity messages to show up in zigbee2mqtt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
problem Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants