Skip to content

Conversation

NathanYee
Copy link

This change is Reviewable

nathan and others added 30 commits March 2, 2016 20:59
added a sandbox folder for experimentation, also added a openCV test …
adding facetracking test code
Added a simple tic tac toe program
integrated openCV and TicTacToe in TicTacToeTest.py
play the game on a timer
@jenwei
Copy link

jenwei commented Apr 4, 2016

@NathanYee + @johnmoreland - Nice job on this.

With regards to the writeup, nice job explaining the implementation, especially with the algorithm. Would have been nice to have more details about how OpenCV comes into play and what you accomplished.

In your reflection, you mentioned some concerns about git merge conflicts. Don't let that hold you back. There are great resources on the course site, and ninjas can also help with Git questions/concerns.

I'll go through your code and add some comments after this.

@@ -0,0 +1,216 @@
import sys, pygame
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't forget to add a header/docstring that gives a brief summary of what the file contains/does.

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.

3 participants