Skip to content
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

Configure a second Elevator motor #29

Open
HexicPyth opened this issue Feb 13, 2019 · 6 comments
Open

Configure a second Elevator motor #29

HexicPyth opened this issue Feb 13, 2019 · 6 comments
Assignees

Comments

@HexicPyth
Copy link
Contributor

No description provided.

@HexicPyth HexicPyth added enhancement New feature or request Probably Intuitive labels Feb 13, 2019
@HexicPyth HexicPyth mentioned this issue Feb 13, 2019
6 tasks
@TheAtomicFire TheAtomicFire self-assigned this Feb 14, 2019
@TheAtomicFire
Copy link

Unfortunately I didn't have enough time at robotics to really understand how the second motor works. So I added another talon and talonWithEncoder that work as inverse to their respective original parts. I based it off the code from last year but I'm not 100% sure that it makes sense

@TheAtomicFire
Copy link

I put the elevator code in my branch in subsystems
I'll also probably look at it again tommorow

@HexicPyth
Copy link
Contributor Author

HexicPyth commented Feb 14, 2019

I'll take a look.
Apparently your branch is missing a lot of RobotMap entries and the whole org.frc3236 package(which contains the encoder Conversion subsystem, amongst other things).
I'll add those to your branch and push it tonight.

Example:

public static int LEFTXBOX = 0; 
public static int RIGHTXBOX = 1; 
public static int LEFTTALONA = 0;
public static int LEFTTALONB = 1; 
public static int RIGHTTALONA = 2; 
public static int RIGHTTALONB = 3;

(Notice there is no mention of Elevators?)

@TheAtomicFire
Copy link

Oh ya thats right I dont think I ever got git to work so the whole branch is outdated. I just added the new elevator code to have it out there, I can try updating the whole branch tommorow but if you want to just push all that than thats really appreciated

@HexicPyth
Copy link
Contributor Author

HexicPyth commented Feb 14, 2019

I happen to be awake so I just merged your branch with the commits you need from vision and got it to build :)

@TheAtomicFire
Copy link

Okay thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants