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

Fixed the seek function #57

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

Conversation

WaseemALTamer
Copy link

Fixed the seek function so it is precise so it dose not round it to the nearest 5 seconds i also added a var for speeding or slowing down the footage

@PaulleDemon
Copy link
Owner

Hey thanks for the pull request, I haven't tested or fully reviewed it yet as I am busy.

However, couple of quick suggestion, remove __pycache__ file add it to .gitignore.

make sure you are following the PEP8 style guidelines, including the naming convention.

I'll free up myself in a month or two, and review the code properly before merging. Till then thank you for your patience and understanding

fixed the time between each frame as it was using sleep.time() which gave me trouble as the videos i was playing was not in the correct frame rate i used a simple module to correct the frame time which takes count of the time code take to run.

i also put the module into the project it is a simple module i wrote
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