Skip to content

Commit 91852f0

Browse files
committed
Rename variable for pulses current hour
1 parent 7cec7b7 commit 91852f0

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

plugwise/nodes/circle.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ def __init__(self, mac, address, message_sender):
7272
self._energy_last_hour_pulses = 0
7373
self._energy_last_local_hour = datetime.now().hour
7474
self._energy_last_populated_slot = 0
75+
self._energy_pulses_current_hour = None
7576
self._energy_pulses_prev_hour = None
7677
self._energy_pulses_today_hourly = None
7778
self._energy_pulses_today_now = None
@@ -80,7 +81,6 @@ def __init__(self, mac, address, message_sender):
8081
self._new_relay_stamp = datetime.now() - timedelta(seconds=MESSAGE_TIME_OUT)
8182
self._pulses_1s = None
8283
self._pulses_8s = None
83-
self._pulses_consumed_1h = None
8484
self._pulses_produced_1h = None
8585
self.calibration = False
8686
self._gain_a = None
@@ -134,8 +134,8 @@ def power_consumption_current_hour(self):
134134
Returns the power usage during this running hour in kWh
135135
Based on last received power usage information
136136
"""
137-
if self._pulses_consumed_1h is not None:
138-
return self.pulses_to_kws(self._pulses_consumed_1h, 3600)
137+
if self._energy_pulses_current_hour is not None:
138+
return self.pulses_to_kws(self._energy_pulses_current_hour, 3600)
139139
return None
140140

141141
@property
@@ -339,8 +339,8 @@ def _response_power_usage(self, message: CirclePowerUsageResponse):
339339
"1 hour consumption power pulse counter for node %s has value of -1, corrected to 0",
340340
self.mac,
341341
)
342-
if self._pulses_consumed_1h != message.pulse_hour_consumed.value:
343-
self._pulses_consumed_1h = message.pulse_hour_consumed.value
342+
if self._energy_pulses_current_hour != message.pulse_hour_consumed.value:
343+
self._energy_pulses_current_hour = message.pulse_hour_consumed.value
344344
self.do_callback(FEATURE_POWER_CONSUMPTION_CURRENT_HOUR["id"])
345345

346346
# Update energy consumption today
@@ -427,7 +427,7 @@ def _collect_energy_pulses(self, start_utc: datetime, end_utc: datetime):
427427
def _calc_todays_energy_pulses(self):
428428
"""Calculate energy consumption today pulse counter"""
429429

430-
if self._pulses_consumed_1h is None:
430+
if self._energy_pulses_current_hour is None:
431431
return None
432432

433433
_current_local_timestamp = datetime.now()
@@ -452,28 +452,28 @@ def _calc_todays_energy_pulses(self):
452452

453453
# Handle post hour rollover, possible due to small clock drifts
454454
if self._energy_last_local_hour != _current_local_hour:
455-
if self._pulses_consumed_1h < self._energy_last_hour_pulses:
455+
if self._energy_pulses_current_hour < self._energy_last_hour_pulses:
456456
_LOGGER.debug(
457457
"energy_consumption_today for %s | post hour rollover reset (hour:%s=>%s, pulses:%s=>%s)",
458458
self.mac,
459459
str(self._energy_last_local_hour),
460460
str(_current_local_hour),
461461
str(self._energy_last_hour_pulses),
462-
str(self._pulses_consumed_1h),
462+
str(self._energy_pulses_current_hour),
463463
)
464464
self._energy_last_local_hour = _current_local_hour
465-
self._energy_last_hour_pulses = self._pulses_consumed_1h
465+
self._energy_last_hour_pulses = self._energy_pulses_current_hour
466466

467467
# Skip pre hour rollover, possible due to small clock drifts
468-
if self._pulses_consumed_1h < self._energy_last_hour_pulses:
468+
if self._energy_pulses_current_hour < self._energy_last_hour_pulses:
469469
_LOGGER.debug(
470470
"energy_consumption_today for %s | Skip for pre hour rollover",
471471
self.mac,
472472
)
473473
self._request_info(self.request_energy_counters)
474474
return None
475475
else:
476-
self._energy_last_hour_pulses = self._pulses_consumed_1h
476+
self._energy_last_hour_pulses = self._energy_pulses_current_hour
477477

478478
# Return current energy counter
479479
if _current_local_hour == 0:

0 commit comments

Comments
 (0)