Skip to content

Commit

Permalink
Restore the update rate of older meters to 1s
Browse files Browse the repository at this point in the history
This is for all meters except the new EM540, which is known
to support a 100ms refresh rate.

In 3248a4 we made the EM24 slower again. This commit extends
that change to all the older meters.

Loosely related to victronenergy/venus#1017
  • Loading branch information
izak committed Mar 27, 2023
1 parent f20222c commit 8099172
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions software/src/ac_sensor.cpp
Expand Up @@ -67,9 +67,11 @@ bool AcSensor::supportMultiphase() const

bool AcSensor::supportFastloop() const
{
// The EM24 appears to jump around if you fast-read it. This logic can be
// improved if more meters need to be added to this list.
return protocolType() != Em24Protocol;
// Only the EM540 really supports the faster updates, and the EM24
// seems particularly jumpy below 20W. Since it adds very little speed
// and sometimes causes problems, including more D-bus traffic, only
// fast-update for the EM540.
return protocolType() == Em540Protocol;
}

void AcSensor::setErrorCode(int code)
Expand Down

0 comments on commit 8099172

Please sign in to comment.