Add IHC8223AL #7494
Add IHC8223AL #7494
Conversation
65698888
commented
May 6, 2024
65698888
commented
|
I refer to the writing method in tuya.ts, why can't mine pass. |
|
Was it in JavaScript format before and not modified to TypeScript format? |
src/devices/honyar.ts
Outdated
| model: 'IHC8223AL', | ||
| vendor: 'Honyar', | ||
| description: 'Smart Power Socket 10A (with power monitoring)', | ||
| fromZigbee: [fz.on_off, fzLocal.honyer_electrical_measurement, fzLocal.honyer_metering], |
There was a problem hiding this comment.
Could you use modernExtend instead here? Example:
There was a problem hiding this comment.
Continue to try to modify, this time there is no error. Please help to check if there are any mistakes.
There was a problem hiding this comment.
I don't see the extend in this PR yet
There was a problem hiding this comment.
Can't I do it the way I'm doing it?
There was a problem hiding this comment.
No, only modernExtend is accepted since this reduces code.
There was a problem hiding this comment.
I use modernExtend for the first time, but I am not sure whether it is successful. Please help to check it.
src/devices/honyar.ts
Outdated
| onOff({powerOnBehavior: false}), | ||
| electricityMeter({cluster: 'honyar'}), | ||
| ], | ||
| onEvent: async (type, data, device) => { |
There was a problem hiding this comment.
Can you try without this? I hope this device support reporting so polling is not needed.
There was a problem hiding this comment.
I've already tried. Proactively report a problem, you have to proactively poll.
There was a problem hiding this comment.
Can you try reusing existing polling code?
Try with:
| onEvent: async (type, data, device) => { | |
| options: [exposes.options.measurement_poll_interval()], | |
| onEvent: (type, data, device, options) => tuya.onEventMeasurementPoll(type, data, device, options), |
src/converters/fromZigbee.ts
Outdated
| exposes.options.calibration('voltage', 'percentual'), exposes.options.precision('voltage'), | ||
| ], | ||
| convert: (model, msg, publish, options, meta) => { | ||
| if (meta.device.dateCode === '20170621') { |
There was a problem hiding this comment.
If I understand correctly, there is a bug for this particular firmware which has the wrong multiplier/divisors?
There was a problem hiding this comment.
Yes, I tried the following method, but it didn't work.
electricityMeter({current: {divisor: 1000}, voltage: {divisor: 1}, power: {divisor: 1}}),
There was a problem hiding this comment.
Did you try to do a reconfigure afterwards? This should work (use the yellow refresh icon on the frontend to do so)
There was a problem hiding this comment.
Thanks for your guidance, the display is normal after using the yellow refresh button on the front.
| @@ -5140,6 +5140,23 @@ const converters2 = { | |||
| return payload; | |||
| }, | |||
| } satisfies Fz.Converter, | |||
| honyer_metering: { | |||
There was a problem hiding this comment.
@65698888 could you check if this one can be removed?
|
@Koenkk Hello, is there anything that needs to be modified? |
|
Yes, please check this comment |
