You’ve connected your physical button so that one side is connected to the digital pin, and the other side is connected to 3.3v
The side of the switch that’s connected to the digital pin is being pulled LOW by the 10k resistor to prevent it floating in an intermediate state (more on that later).
This means that when you press the physical button, the digital pin will be pulled HIGH.
Your code had defined the digital pins for the physical buttons as INPUT_PULLUPs, meaning that they’ll be pulled HIGH by default, so external pull-up resistors aren’t needed to prevent them from floating, and all of your code is looking for a LOW signal from the switch to actuate your relays.
If you throw away the pull-down resistors and connect the other side of the switch to GND rather than 3.3v it should work as desired.
However, you should also read this:
Pete.