-
Notifications
You must be signed in to change notification settings - Fork 30
Python3 and Retina #40
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
Conversation
Oh yeah, Tkinter -> tkinter. |
Removed issues I fixed
1c26f70
to
c62897c
Compare
@JonKoser Hey, thank you for this pull request. I have reviewed it, it works as expected on python 3.x. Do you have any unfinished work before we merge this into master? |
I have another branch which adds a confidence level for OpenCV image recognition, but I can set that up as a separate PR. Thanks for looking over this! |
Alternatively, I could merge it to this. Which do you prefer? |
I have been looking into OpenCV's confidence level also, but if you had done it already - perfect! But yes, that's for another pull request! |
This PR updates the code to Python3 and fixes the retina display issues for OSX.
I haven't written issues for these so if you'd like me to do that first, please let me know.