fixing the check for the phase CT vs. circuit CTs for current scaling

[asolochek]

CTInputPort::C is 0, so the correct current scale factor was never being applied to CT A or B.

[benlbaum]

You're correct here, the code could also explicitly check for A, B, or C phases to be more clear.

const bool is_mains =
(this->input_port_ == CTInputPort::A) ||
(this->input_port_ == CTInputPort::B) ||
(this->input_port_ == CTInputPort::C);

const double scalar = is_mains
? (775.0 / 42624.0) // mains inputs A/B/C
: (775.0 / 170496.0); // branch inputs 1–16

this->current_sensor_->publish_state(raw_current_d * scalar);

@digiblur are you able to merge this PR? Thanks!

[madcatzfight]

Good Catch!

[digiblur]

You're correct here, the code could also explicitly check for A, B, or C phases to be more clear.

const bool is_mains = (this->input_port_ == CTInputPort::A) || (this->input_port_ == CTInputPort::B) || (this->input_port_ == CTInputPort::C);

const double scalar = is_mains ? (775.0 / 42624.0) // mains inputs A/B/C : (775.0 / 170496.0); // branch inputs 1–16

this->current_sensor_->publish_state(raw_current_d * scalar);

@digiblur are you able to merge this PR? Thanks!

Let me try in the morning

[natevw]

Looks like the upstream dev branch now has a similar fix as the one proposed in #1 (comment) as part of the new variant: vue3 support: emporia-vue-local#362 (see the new static inline bool is_mains_port(CTInputPort port) helper).