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

Submitting changes #1

Open
wants to merge 2 commits into
base: release
Choose a base branch
from
Open

Submitting changes #1

wants to merge 2 commits into from

Conversation

johnmoreland
Copy link
Owner

Cleaned up the code.

Also struggled for an hour with trying to implement GIF functionality (via openCV, and an "images2gif.py" script i found and tried to adapt, but PIL was not compliant.)

@@ -1,38 +1,35 @@
""" TODO: Put your header comment here """
"""
Copy link

Choose a reason for hiding this comment

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

Good job addressing the comments, but I don't see any evidence of new functionality.. Even if it didn't work, include what you tried.

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