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

Protocol: Separate ContextHandler instances for running on robot and definition #28

Open
Yuffster opened this issue Aug 15, 2016 · 0 comments
Milestone

Comments

@Yuffster
Copy link
Owner

Yuffster commented Aug 15, 2016

Otherwise if we were to run a Protocol and then keep defining things, the Context would be messed up from the Protocol run.

Not an issue when we replay the entire Protocol definition, but it does become an issue when we're using next to run one step at a time, for example in tests.

Plus the MotorControlHandler can pollute the ContextHandler with things that would put it in a different state than during the definition phase alone.

@Yuffster Yuffster modified the milestones: 1.0, 1.2 Aug 15, 2016
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

No branches or pull requests

1 participant