Skip to content

Commit 1349c1a

Browse files
committed
Add/update cache-related logging
1 parent f18b74a commit 1349c1a

File tree

2 files changed

+20
-4
lines changed

2 files changed

+20
-4
lines changed

plugwise_usb/nodes/circle.py

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,7 @@ async def _calibration_update_state(
269269
self._set_cache(CACHE_CALIBRATION_GAIN_B, gain_b)
270270
self._set_cache(CACHE_CALIBRATION_NOISE, off_noise)
271271
self._set_cache(CACHE_CALIBRATION_TOT, off_tot)
272+
_LOGGER.debug("Saving calibration update to cache for %s", self._mac_in_str)
272273
await self.save_cache()
273274
return True
274275

@@ -563,6 +564,7 @@ async def energy_log_update(self, address: int | None) -> bool:
563564

564565
self._energy_counters.update()
565566
if energy_record_update:
567+
_LOGGER.debug("Saving energy record update to cache for %s", self._mac_in_str)
566568
await self.save_cache()
567569

568570
return True
@@ -782,6 +784,7 @@ async def _relay_update_state(
782784
await self.publish_feature_update_to_subscribers(
783785
NodeFeature.RELAY, self._relay_state
784786
)
787+
_LOGGER.debug("Saving relay state update to cache for %s", self._mac_in_str)
785788
await self.save_cache()
786789

787790
async def _relay_update_lock(self, state: bool) -> None:
@@ -801,6 +804,7 @@ async def _relay_update_lock(self, state: bool) -> None:
801804
await self.publish_feature_update_to_subscribers(
802805
NodeFeature.RELAY_LOCK, self._relay_lock
803806
)
807+
_LOGGER.debug("Saving relay lock state update to cachefor %s", self._mac_in_str)
804808
await self.save_cache()
805809

806810
async def clock_synchronize(self) -> bool:
@@ -1147,6 +1151,7 @@ async def _relay_init_update_state(self, state: bool) -> None:
11471151
await self.publish_feature_update_to_subscribers(
11481152
NodeFeature.RELAY_INIT, self._relay_config
11491153
)
1154+
_LOGGER.debug("Saving relay_init state update to cachefor %s", self._mac_in_str)
11501155
await self.save_cache()
11511156

11521157
@raise_calibration_missing
@@ -1287,7 +1292,7 @@ async def energy_reset_request(self) -> None:
12871292
NO_PRODUCTION_INTERVAL,
12881293
)
12891294
if (response := await request.send()) is None:
1290-
raise NodeError("No response for CircleMeasureIntervalRequest.")
1295+
raise NodeError("No response for CircleMeasureIntervalRequest")
12911296

12921297
if response.response_type != NodeResponseType.POWER_LOG_INTERVAL_ACCEPTED:
12931298
raise MessageError(
@@ -1298,7 +1303,10 @@ async def energy_reset_request(self) -> None:
12981303
# Clear the cached energy_collection
12991304
if self._cache_enabled:
13001305
self._set_cache(CACHE_ENERGY_COLLECTION, "")
1301-
_LOGGER.warning("Energy-collection cache cleared successfully")
1306+
_LOGGER.warning(
1307+
"Energy-collection cache cleared successfully, updating cache for %s",
1308+
self._mac_in_str,
1309+
)
13021310
await self.save_cache()
13031311

13041312
# Clear PulseCollection._logs
@@ -1307,6 +1315,12 @@ async def energy_reset_request(self) -> None:
13071315

13081316
# Request a NodeInfo update
13091317
if await self.node_info_update() is None:
1310-
_LOGGER.warning("Node info update failed after energy-reset")
1318+
_LOGGER.warning(
1319+
"Node info update failed after energy-reset for %s",
1320+
self._mac_in_str,
1321+
)
13111322
else:
1312-
_LOGGER.warning("Node info update after energy-reset successful")
1323+
_LOGGER.warning(
1324+
"Node info update after energy-reset successful for %s",
1325+
self._mac_in_str,
1326+
)

plugwise_usb/nodes/node.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -577,6 +577,7 @@ async def update_node_details(
577577
self._node_info.timestamp = timestamp
578578
self._set_cache(CACHE_NODE_INFO_TIMESTAMP, timestamp)
579579

580+
_LOGGER.debug("Saving Node calibration update to cache for %s" self.mac)
580581
await self.save_cache()
581582
if timestamp is not None and timestamp > datetime.now(tz=UTC) - timedelta(
582583
minutes=5
@@ -654,6 +655,7 @@ async def unload(self) -> None:
654655
return
655656
if self._cache_save_task is not None and not self._cache_save_task.done():
656657
await self._cache_save_task
658+
_LOGGER.debug("Writing cache to disk while unloading for %s" self.mac)
657659
await self.save_cache(trigger_only=False, full_write=True)
658660

659661
def _get_cache(self, setting: str) -> str | None:

0 commit comments

Comments
 (0)