Motor Encoder assistance[Turning]

[ThisIsLeo]

We are currently have an issue with Motor Encoder. This is our first time dealing with Motor Encoders.
When making the Motor Encoder deal with assisting the robot in moving a certain degree, is their a way to make sure it actually stops after doing it’s task?
When we tried getting the correct amount of Turn Ticks, the robot would always move according to the time.
For Example:

autoTurnEncoder(60,90); 
wait1Msec(1000);

If you were to change the time to, (2000) the robot continues to rotate throughout the 2 seconds.
Is this normal?

[MayorMonty]

Make sure you’re turning off your motors after the turn. Also, turning with encoders is suboptimal, and can often lead to a pretty major turn error. If possible, invest in a gyro (which is compatible with both Cortex and V5). Would recommend this article by @biglesliep

[ThisIsLeo]

We do have the motors turned off.

autoTurnEncoder(60,90);
wait1Msec(1000);

motor[port3] = 0;
motor[port5] = 0;
motor[port2] = 0;
motor[port4]= 0;
wait1Msec(1000);

[callen]

@ThisIsLeo

When making the Motor Encoder deal with assisting the robot in moving a certain degree, is their a way to make sure it actually stops after doing it’s task?
When we tried getting the correct amount of Turn Ticks, the robot would always move according to the time.
For Example:

autoTurnEncoder(60,90); 
wait1Msec(1000);

If you were to change the time to, (2000) the robot continues to rotate throughout the 2 seconds.
Is this normal?

It would be good to post the code for autoTurnEncoder().

Get rid of that wait1Msec(1000); entirely. That’s your problem. You finish your turn (theoretically, assuming autoTurnEncoder() works), and then you keep the motors running for another 1000 ms before turning them off.

Inside autoTurnEncoder(int speed, int angle) (guessing at those two), your code should look something like

motor[port3] = speed; // not sure which motors turn together and which opposite
motor[port5] = speed;
motor[port2] = -speed;
motor[port4]= -speed;
while(encoder value is less than desired as calculated here or prior to here) {
wait1Msec(10);
}

[ThisIsLeo]

Here is the code we originally had.

 void autoTurnEncoder(int speed, int degs)
{
nMotorEncoder[port5];
nMotorEncoder[port4];
int ticks = degs * eTurnTicks;

while (abs(nMotorEncoder[port5]) < ticks *.6)
{
	int rDiff = abs(nMotorEncoder[port5]) - abs(nMotorEncoder[port4]);
	int rMod = sgn(rDiff)*speed*.1;

	motor[port3] = speed;
	motor[port5] = speed;
	motor[port2] = -speed - rMod;
	motor[port4] = -speed - rMod;
}

while (abs(nMotorEncoder[port5]) < ticks )
{
int rDiff = abs(nMotorEncoder[port5]) - abs(nMotorEncoder[port4]);
int rMod = sgn(rDiff)speed.1;

	motor[port3] = speed*.8;
	motor[port5] = speed*.8;
	motor[port2] = -speed*.8 - rMod;
	motor[port4] = -speed*.8 - rMod;

	autoTurnTime(-speed/2, 50);

}
}

[callen]

I’m not used to RobotC, but should that first line (and similar with the second line) be this:

nMotorEncoder[port5] = 0;

Anyway, as I said before, you really need to delete that wait1Msec(1000); command entirely. Otherwise your motors are going to keep spinning at speed*0.8 for that whole 1000 ms.

(Not sure what autoTurnTime() is.)