Fixed example sketches, verified that burst mode is working

Also kind of fixed Example3, but it's still kind of borked. Will open an issue to look into it

Author: FischerMoseley

MCP9600_Test/MCP9600_Test.ino

@@ -18,6 +18,7 @@ MCP9600 tempSensor;
 
 void setup(){
     Serial.begin(115200);
+    Wire.begin(10000);
 
     //check if the sensor is connected
     if(tempSensor.isConnected()){

examples/Example1_BasicReadings/Example1_BasicReadings.ino

@@ -23,6 +23,7 @@ thermocoupleType type = TYPE_S; //the type of thermocouple to change to!
 
 void setup(){
     Serial.begin(115200);
+    Wire.begin(10000);
 
     //check that the sensor is connected
     if(tempSensor.isConnected()){

examples/Example2_SetThermocoupleType/Example2_SetThermocoupleType.ino

@@ -19,10 +19,11 @@
 
 #include <SparkFun_MCP9600.h>
 MCP9600 tempSensor;
-uint8_t coeffecient = 5;
+uint8_t coeffecient = 3;
 
 void setup(){
     Serial.begin(115200);
+    Wire.begin(10000);
 
     //check if the sensor is connected
     if(tempSensor.isConnected()){
@@ -47,13 +48,17 @@ void setup(){
     Serial.print(coeffecient);
     Serial.println("!");
 
+    tempSensor.setFilterCoeffecients(coeffecient);
+
     //tell us if the coeffecient was set sucessfully
     if(tempSensor.getFilterCoeffecients() == coeffecient){
         Serial.println("Filter Coeffecients set sucessfully!");
     }
 
     else{
         Serial.println("Setting filter coeffecient failed!");
+        Serial.println("The value of the coeffecient is: ");
+        Serial.println(tempSensor.getFilterCoeffecients(), BIN);
     }
 }
 
@@ -65,6 +70,8 @@ void loop(){ //print the thermocouple, ambient and delta temperatures every 200m
     Serial.print(" °C   Temperature Delta: ");
     Serial.print(tempSensor.tempDelta());
     Serial.print(" °C");
+    Serial.print("   Current Coeffecient: ");
+    Serial.print(tempSensor.getFilterCoeffecients(), BIN);
     Serial.println();
-    while(1); //delay(200);
+    delay(200);
 }

examples/Example3_SetFilterCoeffecients/Example3_SetFilterCoeffecients.ino

@@ -48,6 +48,7 @@ Thermocouple_Resolution thermocoupleRes = RES_14_BIT;
 
 void setup(){
   Serial.begin(115200);
+  Wire.begin(10000);
   //check if the sensor is connected
   if(tempSensor.isConnected()){
       Serial.println("Device will acknowledge!");

examples/Example4_SetResolution/Example4_SetResolution.ino

@@ -8,8 +8,8 @@
   This example configures the shutdown (or "operating") mode that the MCP9600 runs in. Shutdown mode disables all
   power consuming activities on the MCP9600, including measurements, but it will still respond to I2C commands sent
   over Qwiic. Burst mode is similar, where the MCP9600 is shutdown until the Arduino asks it to wake up and take a 
-  number of samples, apply any filtering, update any outputs, and then go back to sleep. This example walks through
-  that process!
+  number of samples, apply any filtering, update any outputs, and then enter shutdown mode. This example walks
+  through that process!
 
   Hardware Connections:
   Attach the Qwiic Shield to your Arduino/Photon/ESP32 or other
@@ -20,10 +20,12 @@
 #include <SparkFun_MCP9600.h>
 MCP9600 tempSensor;
 Shutdown_Mode mode = BURST;
-Burst_Sample samples = SAMPLES_32;
+Burst_Sample samples = SAMPLES_8;
 
 void setup(){
   Serial.begin(115200);
+  Wire.begin(10000);
+  
   //check if the sensor is connected
   if(tempSensor.isConnected()){
     Serial.println("Device will acknowledge!");
@@ -43,8 +45,8 @@ void setup(){
   }
 
   //put the MCP9600 into burst mode!
+  tempSensor.setBurstSamples(samples); 
   tempSensor.setShutdownMode(BURST);
-  tempSensor.setBurstSamples(samples);  
 }
 
 void loop(){
@@ -63,5 +65,4 @@ void loop(){
     //clear the register and start a new burst cycle!
     tempSensor.startBurst();
   }
-  delay(200); //we want to be able to see our data, so we won't let it print any faster than 200ms
 }

examples/Example5_BurstMode/Example5_BurstMode.ino

@@ -91,7 +91,7 @@ bool MCP9600::setAmbientResolution(Ambient_Resolution res){
   bitWrite(config, 7, res); //set the bit that controls the ambient (cold) junction resolution
 
   bool failed = writeSingleRegister(DEVICE_CONFIG, config); //write new config register to MCP9600
-  failed &= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was set properly
+  failed |= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was set properly
   return failed; //return 1 if the write failed or the register wasn't properly set, 0 otherwise
 }
 
@@ -108,7 +108,7 @@ bool MCP9600::setThermocoupleResolution(Thermocouple_Resolution res){
   bitWrite(config, 5, lowResolutionBit); //set 5th bit of config register to 0th bit of the resolution
 
   bool failed = writeSingleRegister(DEVICE_CONFIG, config); //write new config register to MCP9600
-  failed &= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
+  failed |= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
   return failed; //return 1 if the write failed or the register wasn't properly set, 0 otherwise
 }
 
@@ -143,17 +143,27 @@ uint8_t MCP9600::setFilterCoeffecients(uint8_t coeffecient){
   if(coeffecient > 7) return 3; //return immediately if the value is too big
 
   uint8_t config = readSingleRegister(THERMO_SENSOR_CONFIG);
-  config = config >> 3;
-  config = config << 3;
-  config |= coeffecient; //set the necessary bits in the config register
+  bitWrite(coeffecient, 3, bitRead(config, 3));
+  bitWrite(coeffecient, 4, bitRead(config, 3));
+  bitWrite(coeffecient, 5, bitRead(config, 3));
+  bitWrite(coeffecient, 6, bitRead(config, 3));
+  bitWrite(coeffecient, 7, bitRead(config, 3));
 
-  writeSingleRegister(THERMO_SENSOR_CONFIG, config);
-  return;
+  //config = config >> 3;
+  //config = config << 3;
+  //config |= coeffecient; //set the necessary bits in the config register
+
+  return writeSingleRegister(THERMO_SENSOR_CONFIG, coeffecient);
 }
 
 uint8_t MCP9600::getFilterCoeffecients(){
   uint8_t config = readSingleRegister(THERMO_SENSOR_CONFIG);
-  return ((config << 5) >> 5); //clear the non-filter-coeffecients data in the config register
+  uint8_t coeff = 0;
+  bitWrite(coeff, 0, bitRead(config, 0));
+  bitWrite(coeff, 1, bitRead(config, 1));
+  bitWrite(coeff, 2, bitRead(config, 2));
+
+  return coeff; //clear the non-filter-coeffecients data in the config register
 }
 
 bool MCP9600::setBurstSamples(Burst_Sample samples){
@@ -166,7 +176,7 @@ bool MCP9600::setBurstSamples(Burst_Sample samples){
   bitWrite(config, 2, lowResolutionBit); //write 0th bit of samples to 2nd of config
 
   bool failed = writeSingleRegister(DEVICE_CONFIG, config); //write new config register to MCP9600
-  failed &= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
+  failed |= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
   return failed; //return 1 if the write failed or the register wasn't properly set, 0 otherwise
 }
 
@@ -190,7 +200,11 @@ bool MCP9600::burstAvailable(){
 bool MCP9600::startBurst(){
   uint8_t status = readSingleRegister(SENSOR_STATUS);
   bitWrite(status, 7, 0); //clear the 7th bit of the status register, and send over I2C
-  return writeSingleRegister(SENSOR_STATUS, status); //return whether the write was successful
+
+  bool failed =  writeSingleRegister(SENSOR_STATUS, status); //return whether the write was successful
+  failed |= setShutdownMode(BURST);
+
+  return failed;
 }
 
 bool MCP9600::setShutdownMode(Shutdown_Mode mode){
@@ -199,13 +213,16 @@ bool MCP9600::setShutdownMode(Shutdown_Mode mode){
   config |= mode;
 
   bool failed = writeSingleRegister(DEVICE_CONFIG, config); //write new config register to MCP9600
-  failed &= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
+  failed |= (readSingleRegister(DEVICE_CONFIG) != config); //double check that it was written properly
   return failed; //return 1 if the write failed or the register wasn't properly set, 0 otherwise
 }
 
 Shutdown_Mode MCP9600::getShutdownMode(){
   uint8_t config = readSingleRegister(DEVICE_CONFIG);
-  return ((config << 6) >> 6); //clear all bits except the last two and return
+  uint8_t mode = 0;
+  bitWrite(mode, 0, bitRead(config, 0));
+  bitWrite(mode, 1, bitRead(config, 1));
+  return mode; //clear all bits except the last two and return
 }
 
 /*------------------------- Internal I2C Abstraction ---------------- */

src/SparkFun_MCP9600.cpp

@@ -76,9 +76,9 @@ enum Burst_Sample {
 };
 
 enum Shutdown_Mode {
-  NORMAL = 0b00,
-  SHUTDOWN = 0b01,
-  BURST = 0b10,
+  NORMAL = 0x00,
+  SHUTDOWN = 0x01,
+  BURST = 0x02,
 };
 
 class MCP9600{