Some cleanup necessary with grounds. All grounds need to connect to each other -- all ground wires, the metal box, and all switch grounds.
As you have it wired, you are taking cable 3 and 4's ground straight to the respective switch. That is not how grounds are done; the cable onward to the lamp can't ground the switch as it has no connection back to the panel.
I see what look like at many as six "ground screws" in the back of the metal box. You are free to terminate any ground wires there.
The switches themselves can ground through their mounting screws. If you want to run a ground wire from the switch to the box, you can do that too, but it's not necessary given the grounded metal boxes.