Jayson,
Can you give a specific issue here? Aside from combing through code and looking for the possibility of an error, it's tough to give remote help without hearing 'Doc - it hurts when I do THIS' ...
Many folks have issues getting their compressor to run or to switch off due to the coupling of the relay with the pressure switch. Others have issues with getting a piston to move either due to incorrect pneumatic setup (physical issues with what tubes go where) or due to the solenoids which control them not being well understood or hooked up right.
I will say that your use of (4,4) and (3,3) for your Solenoid and Compressor are indeed suspicious though. The Compressor object takes two inputs for the relay and the pressure switch. A solenoid, on the other hand, while capable of taking two inputs is usually done via one input. Unless you're doing something sophisticated, you should have your modules setup in your cRio in the default arrangement - in which case you should be instantiating your solenoid with a single argument for its channel #. By giving two input parameters, you are specifying both the channel and module number which is not likely what you're intention is - in my estimation.
bob