Bituo 3 phase meters missing phase A

[marekwichtowski]

Hey @script0803!

Thanks for adding support for SPM02-3Z3. Looks like it is missing measurements for phase A - would you mind adding?

zigbee-herdsman-converters/src/devices/bituo_technik.ts

Line 217 in ce520ab

zigbeeModel: ["SPM02X001", "SPM02X", "SPM02-3Z3"],

Looks like the 3 phase meter is only exposing 2 phases.
https://www.zigbee2mqtt.io/devices/SPM02-3Z3.html#zemismart-spm02-3z3

[marekwichtowski]

After checking again, it's just that phase A is listed without the "phase_a" for some measurements, but power_factor seems missing for phase a:
e.power_reactive(), e.power_reactive_phase_b(), e.power_reactive_phase_c(), e.power_apparent(), e.power_apparent_phase_b(), e.power_apparent_phase_c(), e.power_factor_phase_b(), e.power_factor_phase_c(),

Can those measurements be named consistently and the missing power factor added?

[script0803]

The parameters like power, current, voltage inside the red box all represent phase a in three-phase products. In the converter, they are directly implemented using the modern extend's **m.electricityMeter**, so there is no need for additional declaration of expose.

[marekwichtowski]

I understand. The definition itself is inconsistent with each parameter defined as parameter, parameter_phase_b, parameter_phase_c. Would changing parameter to parameter_phase_a impact one phase devices? Would adding an extra (duplicate) parameter_phase_a make it worse?