8000 Report 0 for instead of None for zero current and voltage (#1483) · rSffsE/python-kasa@0aa1242 · GitHub
[go: up one dir, main page]

Skip to content

Commit 0aa1242

Browse files
authored
Report 0 for instead of None for zero current and voltage (python-kasa#1483)
- Report `0` instead of `None` for current when current is zero. - Report `0` instead of `None` for voltage when voltage is zero
1 parent 5b9b897 commit 0aa1242

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

kasa/smart/modules/energy.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,15 +126,17 @@ def consumption_total(self) -> float | None:
126126
@raise_if_update_error
127127
def current(self) -> float | None:
128128
"""Return the current in A."""
129-
ma = self.data.get("get_emeter_data", {}).get("current_ma")
130-
return ma / 1000 if ma else None
129+
if (ma := self.data.get("get_emeter_data", {}).get("current_ma")) is not None:
130+
return ma / 1_000
131+
return None
131132

132133
@property
133134
@raise_if_update_error
134135
def voltage(self) -> float | None:
135136
"""Get the current voltage in V."""
136-
mv = self.data.get("get_emeter_data", {}).get("voltage_mv")
137-
return mv / 1000 if mv else None
137+
if (mv := self.data.get("get_emeter_data", {}).get("voltage_mv")) is not None:
138+
return mv / 1_000
139+
return None
138140

139141
async def _deprecated_get_realtime(self) -> EmeterStatus:
140142
"""Retrieve current energy readings."""

0 commit comments

Comments
 (0)
0