Skip to content

Conversation

@kevinfrei
Copy link
Member

Take a look at my changes; Please provide feedback!

// now
public static double MotorResistance = 9.2 / 12;

// TODO: Make this more configurable. Maybe just turn it into a little helper function that
Copy link
Member Author

Choose a reason for hiding this comment

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

This "REGRESSION_*" stuff should be changed to be a function in the Config class, instead of all these constants.

return Command.create(self::setRegressionTeleop);
}

public static Command IncreaseRegressionDTeleop() {
Copy link
Member Author

Choose a reason for hiding this comment

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

This ones feels like a hack for something misbehaving. Let's see if we can fix the misbehavior.

Copy link
Member Author

@kevinfrei kevinfrei left a comment

Choose a reason for hiding this comment

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

Try to move the validator opmode into the component.


@Configurable
@SuppressWarnings("unused")
@TeleOp(name = "Launcher Cfg")
Copy link
Member Author

Choose a reason for hiding this comment

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

Sure would be cool if this were available as part of the component (I should try it)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants