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

Corrected the net logic and fixed an NPE in the Toggle command. #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rhauch
Copy link
Member

@rhauch rhauch commented Sep 13, 2014

These are the changes we made tonight to get the nets to work. Without these changes, when the nets were somewhere between open and closed, pressing the net button would result in an unknown state in the Toggle command and an NPE in the console print statement, making the entire robot inoperable.

This code was tested on a a few commits earlier than the latest master, so strictly speaking this commit against current master is untested. The differences in the 'master' branch are quite minor and unrelated, so I expect it to work. It can be merged after competition, though. (And we'll probably want to tag it after we know it works.)

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.

1 participant