Relé Arduino PIR con interruptor de anulación manual

Creo que su principal problema es que no está asignando una variable o usando una ifdeclaración con su digitalRead(PIR_override);, pero incluso con eso puede perderse el botón presionado en el bucle. Si es un interruptor momentáneo, sube/baja solo durante el tiempo que se presiona, por lo que el procesador tendría que ver esto en el momento adecuado en el bucle while. Puede mantener presionado el botón durante aproximadamente un segundo o puede conectar el botón al pin 2 o 3 y usar una interrupción, que parece que su botón está conectado al pin 3 de acuerdo con su código. Creo que esta es una mejor solución. En el mundo incrustado, las interrupciones son tus amigas.

Entonces, según la documentación ( https://www.arduino.cc/en/Reference/AttachInterrupt ), podría hacer algo como esto:

const byte PIR_override = 3;        // the number of the switch pin
volatile boolean override = false;          // Use this to know if your currently  in override mode or not when looking at the button press

En el interior Setup()agregar:

pinMode(PIR_override, INPUT);
attachInterrupt(digitalPinToInterrupt(PIR_override), motionOverride, LOW); // the sense may need to be changed - from your image it looks like the button goes low when pressed

Así que ahora motionOverridese llamará cuando se presione el botón. Simplemente ponga lo que quiere que suceda dentro de esta función. Algo como esto:

void motionOverride()
{
    if(override) // overriding motion and the button has been pressed - got to PIR mode
    {
        digitalWrite(overridestatus, LOW);  // turn override status LED OFF
        digitalWrite(relaypin, LOW);  // turn relay OFF - you may not want to do this... depends on what you want
        override = false;               // set the override flag
    }
    else // not overriding motion and the button has been pressed - go to override mode
    {
        digitalWrite(overridestatus, HIGH);  // turn override status LED ON
        digitalWrite(relaypin, HIGH);  // turn relay ON
        override = true;                // set the override flag
    }

   // I guess Arduino takes care of clearing the interrupt for you
}

Luego, dentro de su principal, loop()solo mire la bandera de anulación (Nota: no verifiqué ninguna de sus lógicas de detección de movimiento PIR, y con suerte no arruiné sus corchetes allí):

void loop()
{
     if(!override)
     {
         val = digitalRead(inputPin);  // read PIR input value
         if (val == HIGH)  // check if the input is HIGH
         {           
              digitalWrite(relaypin, HIGH);  // turn LED ON
              if (pirState == LOW) 
              {
                   // we have just turned on
                   Serial.println("Motion detected!");
                   // We only want to print on the output change, not state
                   pirState = HIGH;
                   delay(DELVAR);    // maximum delay is 32776 millisecons,
                   //  delay(DELVAR);    // so add multiple delays together to get
                   // delay(DELVAR);    // so add multiple delays together to get
             }
        } 
        else
        {
            digitalWrite(relaypin, LOW); // turn LED OFF
            if (pirState == HIGH)
            {
                 // we have just turned of
                 Serial.println("Motion ended!");
                 // We only want to print on the output change, not state
                 pirState = LOW;
            } 
        }
     }
}

Pruébalo si quieres. Si hay un error, intentaré solucionarlo.

He sugerido una edición de su código; las sangrías no estaban presentes, por lo que las agregué para que las declaraciones if-else anidadas fueran más claras.

Me gustaría abordar algunos problemas que tengo; Cambie las definiciones de sus pines:

int PIR_override = 3;         // the number of the switch pin
int overridestatus = 12;       // the number of the led pin
int relaypin = 13;                // choose the pin for the LED
int inputPin = 2;               // choose the input pin (for PIR sensor)

a algo como:

#define PIR_override 3         // the number of the switch pin
#define overridestatus 12       // the number of the led pin

#define relaypin 13                // choose the pin for the LED
#define inputPin 2               // choose the input pin (for PIR sensor)

Con su uso actual de variables enteras, es posible que su valor cambie, lo que ciertamente no desea hacer. El uso de números enteros para las definiciones de pines también consume espacio en su pila. Puede que esto nunca sea un problema real, pero es algo a considerar.

Se puede encontrar información más detallada sobre las directivas de preprocesador en otros lugares.

Cuando lees tu pin PIR_override:

digitalRead(PIR_override);  // read input value of PIR override switch

El valor no se está utilizando. Debe asignar esto a una variable.

También veo algunos problemas con su lógica if-else. Tome este código por ejemplo:

if (val == LOW)
{            // check if the input is LOW
    val = digitalRead(inputPin);  // read input value

Inicialmente, val está configurado para ser bajo. Está bien. La declaración if será verdadera y se reasignará val. Pero, ¿qué pasa si val se reasigna a ALTO? Una vez que se alcanza el final del ciclo y se reinicia, no hay otra declaración para manejar val == ¡ALTO!

Me parece que lo que está tratando de hacer con su lógica if-else anidada está cerca de una máquina de estado. Si aún no lo ha investigado, sugeriría buscar 'máquinas de estado if-else' o 'máquinas de estado switch-case'.

De lo contrario, si desea continuar con el código que tiene ahora, consideraría escribir los valores de las variables iniciales y "recorrer" manualmente su código.