Skip to content
This repository has been archived by the owner on Nov 4, 2020. It is now read-only.

added subsystem #3

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

added subsystem #3

wants to merge 15 commits into from

Conversation

cocothebam
Copy link

moves backwards and forwards

moves backwords and forwords
Copy link
Member

@KenwoodFox KenwoodFox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work today! Make those code changes when you can.

src/main/java/frc/robot/subsystems/Drivetrain.java Outdated Show resolved Hide resolved
src/main/java/frc/robot/subsystems/Drivetrain.java Outdated Show resolved Hide resolved
added values and Robot Telliop
defined on what the code does (move forwords and nbackwords)
fix problems joe found
Copy link
Member

@KenwoodFox KenwoodFox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks correct! Make sure to resolve the comments on earlier reviews.
Your next step is to call your function from within robot.java, use joe-drivetrain as an example.

cocothebam and others added 5 commits October 9, 2019 16:25
when turns it says TUUUUUUUUUUUUURRRRRRRRRRRRRRRRRRRRRRNNNNNNNNNNNN
and when it moves it says scurt
inverted yaw and inverted control
cocothebam and others added 2 commits October 28, 2019 15:28
added new commands and removed the manual call thought the subsystem
@KenwoodFox
Copy link
Member

You need to test this code first.

Changed pullstring to Turing left and Turning right instead of TUUUUUUUUUUUUUURRRRRRRRRRRRRRRRRRNNNNNN and Scurt
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants