forked from achamaikin/CCSModuleSW30Web
fix meter uint32 wrap causing bogus multi-GWh session energy (v1.0.33).
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
+1
-1
@@ -27,7 +27,7 @@ extern "C" {
|
|||||||
/* USER CODE BEGIN EC */
|
/* USER CODE BEGIN EC */
|
||||||
#define FW_VERSION_MAJOR 1
|
#define FW_VERSION_MAJOR 1
|
||||||
#define FW_VERSION_MINOR 0
|
#define FW_VERSION_MINOR 0
|
||||||
#define FW_VERSION_PATCH 32
|
#define FW_VERSION_PATCH 33
|
||||||
/* USER CODE END EC */
|
/* USER CODE END EC */
|
||||||
|
|
||||||
/* Exported macro ------------------------------------------------------------*/
|
/* Exported macro ------------------------------------------------------------*/
|
||||||
|
|||||||
+11
-6
@@ -20,17 +20,24 @@ void METER_CalculateEnergy() {
|
|||||||
uint32_t currentTick = HAL_GetTick(); // Получаем текущее время в миллисекундах
|
uint32_t currentTick = HAL_GetTick(); // Получаем текущее время в миллисекундах
|
||||||
uint32_t elapsedTimeMs = currentTick - METER.lastTick; // Вычисляем время, прошедшее с последнего вызова в секундах
|
uint32_t elapsedTimeMs = currentTick - METER.lastTick; // Вычисляем время, прошедшее с последнего вызова в секундах
|
||||||
METER.lastTick = currentTick; // Обновляем время последнего вызова для текущего коннектора
|
METER.lastTick = currentTick; // Обновляем время последнего вызова для текущего коннектора
|
||||||
uint32_t energyWs = CONN.Power * elapsedTimeMs / 1000; // Рассчитываем энергию в ватт-секундах за прошедший промежуток времени
|
// uint64: Power*dt can exceed 2^32 at high power / long dt
|
||||||
|
uint64_t energyWs = ((uint64_t)CONN.Power * elapsedTimeMs) / 1000U;
|
||||||
|
|
||||||
//Расчет энергии теперь идет всегда, смещение берем суммарное
|
//Расчет энергии теперь идет всегда, смещение берем суммарное
|
||||||
METER.EnergyPSU_Ws += energyWs;
|
METER.EnergyPSU_Ws += energyWs;
|
||||||
|
|
||||||
// Абсолютное значение энергии разъема складывается из накопленной дробной части и значения со счетчиков
|
// Divide in 64-bit first — casting Ws to uint32 before /3600 truncates ~1193 kWh
|
||||||
METER.AbsoluteEnergy = (uint32_t)METER.EnergyPSU_Ws/3600; //переводим в Вт*час
|
METER.AbsoluteEnergy = (uint32_t)(METER.EnergyPSU_Ws / 3600U);
|
||||||
|
|
||||||
if(METER.enable) {
|
if(METER.enable) {
|
||||||
//enabled state
|
//enabled state
|
||||||
CONN.Energy = METER.AbsoluteEnergy - METER.EnergyOffset;//переводим в Вт*час
|
if (METER.AbsoluteEnergy >= METER.EnergyOffset) {
|
||||||
|
CONN.Energy = METER.AbsoluteEnergy - METER.EnergyOffset;
|
||||||
|
} else {
|
||||||
|
// Guard unsigned underflow (e.g. after Absolute wrap anomaly)
|
||||||
|
CONN.Energy = 0;
|
||||||
|
METER.EnergyOffset = METER.AbsoluteEnergy;
|
||||||
|
}
|
||||||
|
|
||||||
}else{
|
}else{
|
||||||
// Reset statе
|
// Reset statе
|
||||||
@@ -42,5 +49,3 @@ void METER_CalculateEnergy() {
|
|||||||
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Binary file not shown.
Binary file not shown.
+4051
-4046
File diff suppressed because it is too large
Load Diff
+4050
-4045
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user