fix(bq27441): Harmonize temperature() return value with project convention. (#274)

* fix(bq27441): Harmonize temperature() to return Celsius.

* fix(bq27441): Fix Copilot review comments on temperature PR.

Author: nedseb

lib/bq27441/README.md

@@ -110,11 +110,12 @@ bq.soh(SohMeasureType.SOH_STAT)   # Health status bits
 #### Temperature
 
 ```python
-bq.temperature(TempMeasureType.BATTERY)        # Battery temperature
-bq.temperature(TempMeasureType.INTERNAL_TEMP)  # Internal IC temperature
-```
+bq.temperature()                                # Battery temperature in °C (default)
+bq.temperature(TempMeasureType.INTERNAL_TEMP)   # Internal IC temperature in °C
 
-**Note:** Temperature is returned as a raw register value (units of 0.1 K). To convert to °C: `temp_c = raw / 10.0 - 273.15`.
+bq.temperature_k()                              # Battery temperature in Kelvin
+bq.temperature_dk()                             # Battery temperature in decikelvin (raw)
+```
 
 ### Configuration
 
@@ -208,4 +209,4 @@ mpremote mount lib/bq27441 run lib/bq27441/examples/fuel_gauge.py
 * Default design capacity is 650 mAh (configurable via `set_capacity()`).
 * Some configuration operations require entering config mode with `enter_config()`.
 * The device may be sealed; use `unseal()` before advanced configuration.
-* Temperature readings are raw register values in 0.1 K units (see conversion above).
+* `temperature()` returns °C by default. Use `temperature_k()` for Kelvin or `temperature_dk()` for raw decikelvin.

lib/bq27441/bq27441/device.py

@@ -242,15 +242,22 @@ def soh(self, soh_measure_type=SohMeasureType.PERCENT):
         else:
             return soh_status
 
-    # Reads and returns specified temperature measurement
-    def temperature(self, temp_measure_type):
-        temp = 0
+    def _read_temperature_dk(self, temp_measure_type=TempMeasureType.BATTERY):
         if temp_measure_type == TempMeasureType.BATTERY:
-            temp = self.read_word(BQ27441_COMMAND_TEMP)
+            return self.read_word(BQ27441_COMMAND_TEMP)
         elif temp_measure_type == TempMeasureType.INTERNAL_TEMP:
-            temp = self.read_word(BQ27441_COMMAND_INT_TEMP)
+            return self.read_word(BQ27441_COMMAND_INT_TEMP)
+        else:
+            raise ValueError("Unsupported TempMeasureType: {!r}".format(temp_measure_type))
+
+    def temperature(self, temp_measure_type=TempMeasureType.BATTERY):
+        return self._read_temperature_dk(temp_measure_type) / 10.0 - 273.15
+
+    def temperature_k(self, temp_measure_type=TempMeasureType.BATTERY):
+        return self._read_temperature_dk(temp_measure_type) / 10.0
 
-        return temp
+    def temperature_dk(self, temp_measure_type=TempMeasureType.BATTERY):
+        return self._read_temperature_dk(temp_measure_type)
 
     # GPOUT Control Functions
 

tests/scenarios/bq27441.yaml

@@ -90,11 +90,24 @@ tests:
     expect: 25
     mode: [mock]
 
-  - name: "Battery temperature raw register value from mock data"
+  - name: "Battery temperature in Celsius"
     action: script
     script: |
-      from bq27441.device import TempMeasureType
-      result = dev.temperature(TempMeasureType.BATTERY)
+      result = dev.temperature()
+    expect_range: [24.94, 24.96]
+    mode: [mock]
+
+  - name: "Battery temperature in Kelvin"
+    action: script
+    script: |
+      result = dev.temperature_k()
+    expect_range: [298.09, 298.11]
+    mode: [mock]
+
+  - name: "Battery temperature in decikelvin"
+    action: script
+    script: |
+      result = dev.temperature_dk()
     expect: 2981
     mode: [mock]