 }
 
 void pumpOff() {
-  if (!pumpIsOn) {
+  if (pumpIsOn) {
     digitalWrite(pumpPin, LOW);
     Serial << STATMSG << "Turned pump off" << ENDMSG;
   }

  Serial << STATMSG << "Set setpoint to " << setpoint << ENDMSG;
}
    

void setDutyCycle(int value) {
  // Check bounds
  if (value > 100) {

}

void getall_msg() {
  Serial << STATMSG << "All parameters requested" << ENDMSG;
  Serial << GETALLMSG << temperature << "," << dutyCycle << "," \
	 << setpoint << "," << temperaturePID.GetMode() \
	 << "," << pumpIsOn << "," << pValue << "," << iValue  << "," << ENDMSG;
}

void changeSET_msg() {
  /* Serial << STATMSG << "Change in setpoint requested" << ENDMSG; */
  while (cmdMessenger.available()) {
    char buf[350] = {'\0'};
    cmdMessenger.copyString(buf, 350);
