wsen-pads: Adress review comments.

Author: Charly-sketch

lib/wsen-pads/examples/altitude.py

@@ -3,7 +3,8 @@
 from wsen_pads import WSEN_PADS
 
 SEA_LEVEL_PRESSURE = 1013.25 # depends on your location, you can adjust it for better altitude estimation
-EXPONENT = 0.1903 # standard atmosphere exponent for altitude calculation
+# International Standard Atmosphere exponent: 1 / 5.255  
+EXPONENT = 1.0 / 5.255
 
 i2c = I2C(1)
 

lib/wsen-pads/examples/basic_reader.py

@@ -1,4 +1,4 @@
-from machine import I2C, Pin
+from machine import I2C
 from time import sleep
 from wsen_pads import WSEN_PADS
 

lib/wsen-pads/examples/test.py

@@ -8,6 +8,8 @@
     REG_CTRL_2,
     REG_STATUS,
     REG_INT_SOURCE,
+    CTRL1_BDU, 
+    CTRL2_IF_ADD_INC
 )
 
 def print_header(title):
@@ -74,10 +76,10 @@ def test_default_registers(sensor):
     ctrl2 = sensor._read_u8(REG_CTRL_2)
 
     # BDU should be enabled by the driver
-    bdu_ok = bool(ctrl1 & 0x02)
+    bdu_ok = bool(ctrl1 & CTRL1_BDU)
 
     # IF_ADD_INC should be enabled by the driver
-    if_add_inc_ok = bool(ctrl2 & 0x10)
+    if_add_inc_ok = bool(ctrl2 & CTRL2_IF_ADD_INC)
 
     if bdu_ok:
         print_pass("BDU enabled")

lib/wsen-pads/wsen_pads/device.py

@@ -1,4 +1,4 @@
-from utime import sleep_ms, ticks_ms, ticks_diff
+from time import sleep_ms, ticks_ms, ticks_diff
 
 from wsen_pads.const import *
 from wsen_pads.exceptions import *
@@ -54,7 +54,8 @@ def __init__(self, i2c, address=WSEN_PADS_I2C_DEFAULT_ADDR):
     def _is_present(self):
         """Return True if the device address is visible on the I2C bus."""
         try:
-            return self.address in self.i2c.scan()
+            self.i2c.readfrom(self.address, 1)  
+            return True  
         except Exception:
             return False
 
@@ -254,14 +255,19 @@ def temperature(self):
         """
         return self.temperature_raw() * TEMPERATURE_C_PER_DIGIT
 
-    def read(self):
+    def read(self, low_noise=False):
         """
         Read and return both pressure and temperature.
+        Has to be used in one-shot mode. Continuous mode must be disabled first.
 
         Returns:
             tuple: (pressure_hpa, temperature_c)
         """
-        self.trigger_one_shot()
+        if continuous_mode := (self._read_u8(REG_CTRL_1) & CTRL1_ODR_MASK) != 0:
+            raise WSENPADSInvalidMode(
+                "read() cannot be used in continuous mode. Please call power_down() first."
+            )
+        self.trigger_one_shot(low_noise=low_noise)
         return self.pressure(), self.temperature()
 
     # ---------------------------------------------------------------------
@@ -302,7 +308,7 @@ def read_one_shot(self, low_noise=False):
         Returns:
             tuple: (pressure_hpa, temperature_c)
         """
-        return self.read()
+        return self.read(low_noise=low_noise)
 
     # ---------------------------------------------------------------------
     # Continuous mode