zigbee-herdsman-converters icon indicating copy to clipboard operation
zigbee-herdsman-converters copied to clipboard

add pulse configuration for Frient Electricity Meter Interface 2

Open ywaf opened this issue 1 year ago • 5 comments

should fix https://github.com/Koenkk/zigbee-herdsman-converters/commit/b7b678862a3871b53606022daf6bd1e1ad8f3e06#commitcomment-133972455

(note: ive never written a converter, might be wrong way to do it)

ywaf avatar Dec 04 '23 02:12 ywaf

Sorry for being inactive here, been a bit busy, is there any docs for this "modernExtend"? I'm complete shit at javascript so not 100% sure exactly what needs to be modified :)

ywaf avatar Dec 13 '23 10:12 ywaf

Not yet because it is extremely new (2 weeks old now) but basically you can drop the expose and to/fromZigbee converters when using:

extend: [
            numeric({
                name: 'pulse_configuration',
                valueMin: 0,
                valueMax: 65535,
                valueStep: 1,
                scale: 1,
                unit: '',
                cluster: 'seMetering',
                attribute: 'develcoPulseConfiguration',
                description: 'Pulses per kwh. Default 1000 imp/kWh. Range 0 to 65535',
            }),
]

Koenkk avatar Dec 13 '23 18:12 Koenkk

If I would like to contribute to the creation of this updated converter for the Meter interface, is this possible? How can I test the converter before pushing? I've tried to add it as an external converter in zigbee2mqtt (in home assistant) but since there is already a converter for the meter it seems zigbee2mqtt always uses the internal converter over the external? Do I need to recompile zigbee2mqtt to test the changes in this PR?

TheMugge avatar Dec 25 '23 14:12 TheMugge

but since there is already a converter for the meter it seems zigbee2mqtt always uses the internal converter over the external?

Z2M always prefers external converters over internal ones.

Koenkk avatar Dec 26 '23 09:12 Koenkk

I can confirm this. Maybe there is a typo in the config?

Hopefully this PR will be released soon, so that might fix it for you as well.

dawiinci avatar Dec 26 '23 17:12 dawiinci