Default code and the Compressor

[Matthew_H]

here is both this years default code (which doesn't work for us), and the default code from last year which does work.

[Bleric]

Well, the code you posted should work. Strange...

Maybe you're reassigning relay8 in a different file, like user_routines_fast? Might do a search for relay8 in some other files, and see if you're setting it equal to something.

[Matthew_H]

We havent changed any of the other files, yet.

[Dave Scheck]

I diffed your 2005 code to the Default code released by FIRST and, as expected, it is identical.

This makes me believe that your compressor and/or pressure switch is wired incorrectly.

Try making the following modification and see what you see

 
relay5_rev = p1_sw_aux2;
relay6_fwd = p3_sw_aux1;
relay6_rev = p3_sw_aux2;
relay7_fwd = p4_sw_aux1;
relay7_rev = p4_sw_aux2;
relay8_fwd = !rc_dig_in18;  /* Power pump only if pressure switch is off. */
relay8_rev = 0;
   
/******************/
/* THIS CODE IS ADDED */
printf("Switch %d  Relay  %d\r",(int)rc_digin18,(int)relay8_fwd);
/******************/

   /*---------- PWM outputs Limited by Limit Switches  ------------------------*/

In the case that the switch always shows 0, your compressor is wired incorrectly. If the relay is showing green, doublecheck the compressor's connection to it. If the relay doesn't change color, check the PWM connection between the relay and relay port 8.

If you see that the switch is always reading 1, then your compressor will never turn on. If that is the case, your switch may be wired incorrectly or defective. To verify this, remove the ! from the line that sets relay8_fwd. If the compressor runs and shuts off at 115 psi, then there's some weirdness on why your switch is returning an inverted value. If the compressor never shuts off, doublecheck all connections between the switch and the RC.

Hopefully that helps...

[Matthew_H]

It DOES work with the 2004 code. but does not work with the 2005. same wiring for both. but we cant find any real diffrence between them other than the camera stuff.

[Greg Ross]

Originally Posted by Bleric

Maybe you're reassigning relay8 in a different file, like user_routines_fast? Might do a search for relay8 in some other files, and see if you're setting it equal to something.

The version of user_routines_fast I just looked at DOES set relay8 = 0 inside of user_routines_fast.

[Mike Betts]

Matthew, et al,

I was planning to hook up a compressor this afternoon anyway. I will try the 2005 default code and report my observations.

Mike

[Matthew_H]

Originally Posted by Mike Betts

Matthew, et al,

I was planning to hook up a compressor this afternoon anyway. I will try the 2005 default code and report my observations.

Mike

We finally got it to work. We changed this the code below and it worked. Please explain why this worked.

relay8_fwd = !rc_dig_in18;  /* Power pump only if pressure switch is off. */
relay8_rev = 0;

relay8_rev = !rc_dig_in18;  /* Power pump only if pressure switch is off. */
relay8_fwd = 0;

[Mike Betts]

Originally Posted by Matthew_H

We finally got it to work. We changed this the code below and it worked. Please explain why this worked.

relay8_fwd = !rc_dig_in18; /* Power pump only if pressure switch is off. */
relay8_rev = 0;

Matthew,

relay8_rev must have been initialized to 1 (or set to 1) somewhere in the code or possibly it was never initialized (an uninitialized variable is, by definition, undefined).

Anyway, congrats on getting it working...

Mike

[steven114]

Keep in mind that you're not supposed to run the compressor backwards - make sure that, if you have the spike in reverse, you switch the leads on the compressor or something.

[stephenthe1]

yeah, mine won't work either. much more importantly, 2.4 won't even take input from our joysticks, yet 2.2 will. these are both unmodified. any suggestions?