[Sean1038]

1) my team actually put the startcompressor.vi in teleop.vi since teleop is in a loop and it continuously runs instead of begin which runs once.
2) I would consider using the solenoid vis instead of the relays because you don't have to go through a spike.
3) you might have to experiment with the values you are sending to your solenoids-- our solenoid actuated with the values of "off" and "forward"
4) you might want to use a latch to control your solenoids because right now it only actuates while you're pressing button 2.