-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tank Drive Cheesy Joysticks swapped? #56
Comments
Is this the method in TankDrive to which you're referring?
Note that this was derived from Team 254's CheesyDriveHelper, and our method's logic and even parameter order seems to follow that of Team 254's. Can you double check that you've not made a mistake elsewhere in your code? |
Yes that is the method. I just double checked, and it does seem that passing it the wheel then throttle works. We can post our usage of the code if you would like. |
@EricLiu2000: I've looked at the code again in more depth, and I'm surprised to hear that the inputs are indeed switched. The existing logic just won't work if the That suggests to me that there is still a problem elsewhere in the code (including Strongback) or in the mapping between the physical devices and the software components on your robot. If you post your code, we'll go through it to see if we can find anything and (if time allows) test it on one of our team's robots. However, I'd ask that you double-check the physical mapping as well. If you're using two separate devices, double check that the port numbers in the code match the physical devices. Or, if you're using a single device, make sure that you're using the correct joystick implementation for your hardware device -- different joysticks are notorious for having different mappings between the buttons/controls and axes. |
https://github.com/team8/jFreyja/tree/strongback is the code we were running. We double checked the physical mapping in the driver station software, our joysticks were mapped to the ports we expected. Note that our code currently compensates for the swap. |
@niharmitra Okay, I've looked at the code again and I'm still not convinced the behavior is as you describe. If you look closely at the logic, you'll see that the Are you sure that you didn't accidentally reverse the inputs? That is by far the most likely cause. |
added maybewaitfor and logged time it took to apply state
It appears according to the documentation, the first joystick is the throttle and the second is the wheel. But from my team's testing it was the other way around?
The text was updated successfully, but these errors were encountered: