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

add video element to readme.md #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

wis
Copy link

@wis wis commented May 31, 2024

The readme currently links to a page on the autopy website, the page embeds a Flash video player, which modern browser can not play.

chrome_FY0Gv7qyJu

I downloaded the video and uploaded it to GitHub through the issue markdown editor and copied the link that the issue editor created to use for the video element.
The video was in a mov container and I converted to a mp4 container using ffmpeg, with -crf 15 so the video is not compressed again, so it retains its image quality.

ffmpeg -i sine-move-mouse\ \[sine-move-mouse\].mov -crf 15 sine-move-mouse\ \[sine-move-mouse\].mp4

The video is only 136KB by the way, insignificantly small.

result:
chrome_WtaT8lprWx

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.

1 participant