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.

Sign up for GitHub

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

Jump to bottom

All control strats should be made in RobotMap, not each time they're used #116

Open
met4000 opened this issue Feb 13, 2019 · 1 comment
Open

All control strats should be made in RobotMap, not each time they're used #116

met4000 opened this issue Feb 13, 2019 · 1 comment
Assignees
@met4000
Labels
5333 Only Only in use by 5333 Available The author wants someone else to look at this enhancement New feature or request
Milestone
Week 6

Comments

@met4000
Copy link
Contributor

met4000 commented Feb 13, 2019

See title.
@met4000 met4000 added enhancement New feature or request 5333 Only Only in use by 5333 Available The author wants someone else to look at this labels Feb 13, 2019
@met4000 met4000 added this to the Week 6 milestone Feb 13, 2019
@met4000 met4000 self-assigned this Feb 13, 2019
@JaciBrunning
Copy link
Member

Not necessarily. Strats like PID for the elevator can't be reused, so they must be reconstructed with a parameter (in the case of the elevator, the height is given in the constructor)

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

@met4000 met4000

Labels
5333 Only Only in use by 5333 Available The author wants someone else to look at this enhancement New feature or request
Projects
None yet
Milestone
Week 6
Development

No branches or pull requests
2 participants
@JaciBrunning @met4000