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

Fix DJI Tello video example #633

merged 1 commit into from Nov 30, 2018


None yet
2 participants
Copy link

eleniums commented Oct 12, 2018

This change allows mplayer to keep up with the video frame events. With the fps set to 25, the video frame events were stacking up and the video was lagging horribly behind.

Change fps to 60
This allows mplayer to keep up with the VideoFrameEvents.

This comment has been minimized.

Copy link

codecov bot commented Oct 12, 2018

Codecov Report

Merging #633 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #633   +/-   ##
  Coverage   77.72%   77.72%           
  Files         157      157           
  Lines       10673    10673           
  Hits         8296     8296           
  Misses       1929     1929           
  Partials      448      448

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0f5ba5...64456b7. Read the comment docs.


This comment has been minimized.

Copy link

trevrosen commented Nov 30, 2018

@eleniums Thanks for the contribution! Finally reviewing here in a minute... 😸

@trevrosen trevrosen merged commit ac0cad7 into hybridgroup:dev Nov 30, 2018

4 checks passed

codecov/patch Coverage not affected when comparing d0f5ba5...64456b7
codecov/project 77.72% remains the same compared to d0f5ba5
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed

@eleniums eleniums deleted the eleniums:fix/fps-tello-video-example branch Nov 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.