Updates from PR feedback + build settings

Author: tyeth

.vscode/settings.json

@@ -3,6 +3,7 @@
         "limits": "c",
         "type_traits": "c"
     },
-    "C_Cpp.dimInactiveRegions": false,
-    "dotnet.defaultSolution": "disable"
+    "C_Cpp.dimInactiveRegions": true,
+    "dotnet.defaultSolution": "disable",
+    "cmake.configureOnOpen": false
 }

src/components/i2c/WipperSnapper_I2C.cpp

@@ -389,14 +389,14 @@ bool WipperSnapper_Component_I2C::initI2CDevice(
   } else if (strcmp("nau7802", msgDeviceInitReq->i2c_device_name) == 0) {
     _nau7802 = new WipperSnapper_I2C_Driver_NAU7802(this->_i2c, i2cAddress);
     if (!_nau7802->begin()) {
-      WS_DEBUG_PRINTLN("ERROR: Failed to initialize NAU7802/303");
+      WS_DEBUG_PRINTLN("ERROR: Failed to initialize NAU7802");
       _busStatusResponse =
           wippersnapper_i2c_v1_BusResponse_BUS_RESPONSE_DEVICE_INIT_FAIL;
       return false;
     }
     _nau7802->configureDriver(msgDeviceInitReq);
     drivers.push_back(_nau7802);
-    WS_DEBUG_PRINTLN("NAU7802/303 Initialized Successfully!");
+    WS_DEBUG_PRINTLN("NAU7802 Initialized Successfully!");
   } else if (strcmp("sgp30", msgDeviceInitReq->i2c_device_name) == 0) {
     _sgp30 = new WipperSnapper_I2C_Driver_SGP30(this->_i2c, i2cAddress);
     if (!_sgp30->begin()) {

src/components/i2c/drivers/WipperSnapper_I2C_Driver_NAU7802.h

@@ -19,6 +19,8 @@
 #include "WipperSnapper_I2C_Driver.h"
 #include <Adafruit_NAU7802.h>
 
+#define NAU7802_TIMEOUT_MS 250 // Timeout waiting for data from NAU7802
+
 /**************************************************************************/
 /*!
     @brief  Class that provides a driver interface for the NAU7802.
@@ -55,20 +57,23 @@ class WipperSnapper_I2C_Driver_NAU7802 : public WipperSnapper_I2C_Driver {
       @returns  True if initialized successfully, False otherwise.
   */
   /*******************************************************************************/
-  bool begin() {
-    if (!_nau7802->begin(_i2c))
-      return false;
+  bool begin() { return _nau7802->begin(_i2c) && configure_nau7802(); }
 
+  /*******************************************************************************/
+  /*!
+      @brief    Configures the NAU7802 sensor.
+      @returns  True if configured successfully, False otherwise.
+  */
+  /*******************************************************************************/
+  bool configure_nau7802() {
     if (!_nau7802->setLDO(NAU7802_3V0)) {
       WS_DEBUG_PRINTLN("Failed to set LDO to 3V0");
+      return false;
     }
 
     if (!_nau7802->setGain(NAU7802_GAIN_128)) {
       WS_DEBUG_PRINTLN("Failed to set gain to 128");
-    }
-
-    if (!_nau7802->setRate(NAU7802_RATE_10SPS)) {
-      WS_DEBUG_PRINTLN("Failed to set sample rate to 10SPS");
+      return false;
     }
     return true;
   }
@@ -86,8 +91,8 @@ class WipperSnapper_I2C_Driver_NAU7802 : public WipperSnapper_I2C_Driver {
 
     // Wait for the sensor to be ready
     while (!_nau7802->available()) {
-      if (millis() - start > 1000) {
-        WS_DEBUG_PRINTLN("NAU7802 not available");
+      if (millis() - start > NAU7802_TIMEOUT_MS) {
+        WS_DEBUG_PRINTLN("NAU7802 data not available");
         return false;
       }
     }