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

Motors and Touch Sensors #6

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

Conversation

ljcrapo
Copy link

@ljcrapo ljcrapo commented Feb 4, 2016

Motors

I added to what was already there and it is a more complete implementation of what is available with the underlying API.

Touch Sensor

This was almost a complete rewrite. If you want it in a separate pull request I can do that as well, it would just be more work. The original functionality was almost completely useless to me.

@ldmberman
Copy link
Owner

Hi @antosop,

Thank you for your work. I think, I will look closer at it tomorrow or on the weekend.

@ljcrapo
Copy link
Author

ljcrapo commented Feb 5, 2016

In trying to use the touch sensor, I've discovered some cases where It is too complicated for simple use. I think you should hold off on this pull request until I push my most recent changes to touch sensor.

@ldmberman
Copy link
Owner

Ok, I'll wait until you come up with the final version.

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.

None yet

2 participants