-
Notifications
You must be signed in to change notification settings - Fork 291
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
Example Python file and README Update #90
base: develop-0.7.0
Are you sure you want to change the base?
Conversation
A function was added to keyboard_and_video.py (get_file_path), that creates a file path based on the OS you're running on. Recording and image taking was updated to use these functions.
Instruction for installing mplayer in 3 major environments was added, along with placing an example for using video_and_keyboard.py.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's looking good :D
Any thoughts hanyazou?
sry but I tried your "made_keyboard_and_video.py platform neutral" version for Ubuntu 18.04 incl. your advices from readme.txt (installing the player) but it did not work. still black screen with battery status etc. on the upper left. do you have any tipps for me? |
Hi, I have the same problem on Windows10. Have you solved it? |
Hi hanyazou,
I am a very big fan of your work, and I am very thankful for your Python Tello client :D !!
With a lot of people having questions about mplayer and video_and_keyboard.py, I wanted to contribute solutions that have helped me and others who have reported similar issues.
First, with people typically using video_and_keyboard.py first, I made the example code platform independent for when users want to save an image or video.
Lastly, since all video examples assume mplayer is installed, I added sections to help users install it based on each major platform.
Hope all is well during this time!
Best,
Walt