!HELP! IM STUCK IN A SUBROUTINE!!! !HELP!

[Daisy]

After running it in my head (which may not be accurate), I think time is the problem. You're incrementing time, so that after 49, it reaches 50, but at it, you do not define what it does with it (ie Left() or Stop(), etc), so it will keep repeating left.

Daisy

[Dave Scheck]

Originally Posted by Daisy

After running it in my head (which may not be accurate), I think time is the problem. You're incrementing time, so that after 49, it reaches 50, but at it, you do not define what it does with it (ie Left() or Stop(), etc), so it will keep repeating left.Daisy

Yeah, but time gets incremented outside of the if statements. Unless he is clearing the PWM values somewhere, whatever was set in Left will still be sent to the motors for an additional cycle. If he is clearing the values, there would be a 1 cycle pause in the PWM output.

I agree with what was said above. Most likely the time variable isn't global or statically defined. Debug prints should make this apparent rather quickly if this is the case.

[cdennisxlx2]

Time is not the problem, that part works great. All i need to know is did i code the subroutines wrong or am i calling them wrong? If im doing those right then why is the code stuck in Left() when i run the robot in autonomous mode?

[Mark McLeod]

Post a zip of your user_routines_fast.c here and we'll tell you.

Right now it looks like you're taking something for granted when you post the problem and not bothering to tell us about some seemingly unimportant detail. In programming all details are important. If you weren't overlooking something your code would be working. As with most bugs you don't know what the important detail is yet.

[Alan Anderson]

Originally Posted by ghhs_1527

Time is not the problem, that part works great.

How do you know that? My first guess is that you're "initializing" the time variable every time through the loop, so it is always equal to zero when you hit the if statement.

Speaking of "the loop", what is causing it to run 26 times per second? We definitely need to see more of the code in order to help you.

[Stuart]

ok its now 5pm and Im awake

ok 50% chance your resetting time to be = to 0 every time you loop.

30% chance that that loop isnt happening every 26 secs(I know it says "im fast I run every 26.1 secs" but its a liar liar pants on fire)

20% chance its something else

[cdennisxlx2]

I know it works because it works in all the other modes, just not the one with the subroutines, we have a dip switch to choose autonomous modes. but here are the user_routines_fast.c and user_routines.h files (attached)

[Alan Anderson]

	if (rc_dig_in11 = 1) /* 1st Autonomy code */

There's your problem right there. You just tried to assign the value 1 to the variable rc_dig_in11. To test for equality, you must use the == symbol, with two equal signs.

The code you posted will always run the BASH routine. You need to change all those single = to double == in your if and else if statements.

[JamesBrown]

Originally Posted by Alan Anderson

	if (rc_dig_in11 = 1) /* 1st Autonomy code */

There's your problem right there. You just tried to assign the value 1 to the variable rc_dig_in11. To test for equality, you must use the == symbol, with two equal signs.

The code you posted will always run the BASH routine. You need to change all those single = to double == in your if and else if statements.

Yep, that would be it.

As a note to any one else who ever needs help the more code you post the better we can help, don't worry about people stealing it, it wont work on our robots anyway.

James

[Dave Scheck]

I agree with Alan and James that a problem exists in the use of the assignment operator, but the initial problem stated that the code was stuck in Left. I don't see how this could be reached. How was it determined that it was getting stuck in Left?

[cdennisxlx2]

i had a student telling me what directions the wheels were turning and i took his word for it instead of double checking, but thank you all for the help

[Matt Krass]

I still think we need to see the rest of the code, or at least how the time variable is defined. If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted. So if it's at the top of your function, and this function runs over and over every loop, it will be reinitialized every loop to 0, no matter how much you increment it. Or, time can be a global and it will retain its value, but if you're assigning 0 to it every loop that will also ruin your day. I think you have a pretty simple (and somewhat hard to notice) problem that I'm unfortunately very familiar with, as Mark McLeod can tell you, how many times did I hit Chris with the robot again?

[cdennisxlx2]

the .c file is attached above, but i know for a fact thats not the problem because ive defined "time" and incrimented it the same way for 3 years now and i have had no problem with that at all.

[Dave Flowerday]

Originally Posted by Matt Krass

I still think we need to see the rest of the code, or at least how the time variable is defined. If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted.

It's there, line 25 of user_routines_fast.c, as a global, so it should be fine. I'm not exactly sure what you mean by "If it isn't defined static, the variable will be re-initialized everytime its host entity is restarted."