Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Python 3 compatilbity #18

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

Closed
wants to merge 0 commits into from
Closed

Python 3 compatilbity #18

wants to merge 0 commits into from

Conversation

hamsolo474
Copy link
Contributor

Hey,

I have ported your excellent library to python 3 while maintaining backwards compatibility to python 2.7, I have also fixed some spelling mistakes.
feel free to hit me up at [email protected] if you have any questions

@thearn
Copy link
Owner

thearn commented Oct 27, 2014

Excellent! I'm going to give your branch a test drive a bit before merging, but I appreciate the enhancements!

@hamsolo474
Copy link
Contributor Author

Hey man,

I dont know if you have had time to give my lib a thorough testing, but i have done a major update that fixes a few of the bugs in the last one, specifically in the servos class.

I have also added strict error checking which allows this api to return useful error messages, so if you try to digitalWrite(13, 'HIGH') without first setting pinMode(13, 'OUTPUT') it throws an error it will also throw errors if you try to digitalWrite(13, 'Apples'). I have added some tests to work test the new functionality.

@thearn
Copy link
Owner

thearn commented Feb 5, 2015

Coming back around to checking this out (since Python 3 compatibility would be great), it looks like there is a conflict between our branches though. Can you resolve this?

@hamsolo474 hamsolo474 closed this Jun 7, 2023
@hamsolo474 hamsolo474 force-pushed the master branch 2 times, most recently from 0d97a84 to 610171b Compare June 7, 2023 10:27
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