add explicit checks to avoid potential div. by zero on application of artificially increased power limit if channel power becomes zero
This commit is contained in:
parent
cd4a327671
commit
b79619bf8b
@ -224,17 +224,20 @@ void PowerLimiterClass::setNewPowerLimit(std::shared_ptr<InverterAbstract> inver
|
|||||||
}
|
}
|
||||||
MessageOutput.printf("[PowerLimiterClass::loop] Limit Non-Persistent: %d W\r\n", newPowerLimit);
|
MessageOutput.printf("[PowerLimiterClass::loop] Limit Non-Persistent: %d W\r\n", newPowerLimit);
|
||||||
|
|
||||||
|
int32_t effPowerLimit = newPowerLimit;
|
||||||
std::list<ChannelNum_t> dcChnls = inverter->Statistics()->getChannelsByType(TYPE_DC);
|
std::list<ChannelNum_t> dcChnls = inverter->Statistics()->getChannelsByType(TYPE_DC);
|
||||||
int dcProdChnls = 0, dcTotalChnls = dcChnls.size();
|
int dcProdChnls = 0, dcTotalChnls = dcChnls.size();
|
||||||
for (auto& c : dcChnls) {
|
for (auto& c : dcChnls) {
|
||||||
if (inverter->Statistics()->getChannelFieldValue(TYPE_DC, c, FLD_PDC) > 0) {
|
if (inverter->Statistics()->getChannelFieldValue(TYPE_DC, c, FLD_PDC) > 1.0) {
|
||||||
dcProdChnls++;
|
dcProdChnls++;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
int32_t effPowerLimit = round(newPowerLimit * static_cast<float>(dcTotalChnls) / dcProdChnls);
|
if (dcProdChnls > 0) {
|
||||||
uint16_t inverterMaxPower = inverter->DevInfo()->getMaxPower();
|
effPowerLimit = round(newPowerLimit * static_cast<float>(dcTotalChnls) / dcProdChnls);
|
||||||
if (effPowerLimit > inverterMaxPower) {
|
uint16_t inverterMaxPower = inverter->DevInfo()->getMaxPower();
|
||||||
effPowerLimit = inverterMaxPower;
|
if (effPowerLimit > inverterMaxPower) {
|
||||||
|
effPowerLimit = inverterMaxPower;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
inverter->sendActivePowerControlRequest(Hoymiles.getRadio(), effPowerLimit, PowerLimitControlType::AbsolutNonPersistent);
|
inverter->sendActivePowerControlRequest(Hoymiles.getRadio(), effPowerLimit, PowerLimitControlType::AbsolutNonPersistent);
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user