check code

[zip100]

I just want you guys to review a code that a teammate wrote. He told me to write that added to the code is allowing the motor to spin at half speed if button 1 is pressed.

[Vikesrock]

This code currently does not do anything different when the button is pressed. You want the motor to run at half speed or half the speed that the joystick commands when the button is pressed? The following assumes that you wanted the first option.

The 0.5 constant in the case statement is not connected to anything.

For the false case of each case block the joystick axis value should pass through the case block.

For the true case the 0.5 constant should be wired to the small orange block on the right edge of the case that will appear when you wire the joystick value through the false case.

[zip100]

edited to what I think you said.

[Vikesrock]

Originally Posted by zip100

joystick axis value meaning ? the axis (y)

Correct, the orange wire coming out of each block that says "axis 2(y)" should pass through the false case instead of over the case block.

[zip100]

after going through the case block, does it go to the tank drive vi

[Vikesrock]

Oops, looks like a forgot to ask something on my first pass.

Do you want both motors to run at 1/2 speed by pushing one trigger (button 1)?

Or do you want each side to run at 1/2 speed when the trigger is pressed on that sides joystick?

The second one is what the code currently does. So ,for example, if the left trigger is pressed the left side will run at a constant .5 and the right side will still run according to the joystick. If both triggers are pressed, both sides will run at constant speed.

Is that what you wanted?

[zip100]

wanted to both joysticks pressed for half speed. So does the second pic look okay.

[Vikesrock]

Originally Posted by zip100

wanted to both joysticks pressed for half speed. So does the second pic look okay.

Yes, if you want each joystick and button #1 to control each side separately I believe the fixed code is correct.

[zip100]

so i do not have to put in axis (y) in the true, only in the false. Also only one end is go9ng to the case structure of the constant. Does that look okay.

[Vikesrock]

Correct, when the button is pressed (true case) you want the motor to run at a constant speed so you do not need the joystick value.

The constant originates in the case so it only goes out, no need for anything to come in.

[zip100]

Thank you for the help, will test out tomorrow and reply on how it went.

[Vikesrock]

If you need additional assistance and would like something more real-time, you can use the AIM, MSN, or Yahoo Messenger info in my profile or contact me on Gtalk (My gmail address is the same as my username here).

I'm not sure if I'm heading out to our shop tomorrow or not, so I may or may not be available by the above methods tomorrow afternoon/evening.

[zip100]

I tried the revised code but it shows error on build process of a broken vi.
Can I get a second opinion? Also the Ethernet cable going from the classmate will not click into the Ethernet port of the classmate it just falls out. Anyone else having these problems

[DonRotolo]

The first problem, I can't help with.

The ethernet port: If another cable also doesn't click, look carefully at the classmate's connector, make sure it's not broken. If another cable clicks & locks, then it was the cable end.

[Vikesrock]

Originally Posted by zip100

I tried the revised code but it shows error on build process of a broken vi.

Could be the floating multiply block on the left side of the picture.

You can click the broken grey arrow at the top of the block diagram and it will pull up a list of errors. Double clicking on an error in the list will highlight the offending VI.