Skip to content
This repository has been archived by the owner on Jan 21, 2022. It is now read-only.

Add dive information in video #122

Closed
patrickelectric opened this issue Nov 12, 2017 · 5 comments
Closed

Add dive information in video #122

patrickelectric opened this issue Nov 12, 2017 · 5 comments
Labels

Comments

@patrickelectric
Copy link
Member

patrickelectric commented Nov 12, 2017

It's possible to use textoverlay to add log information with video while recording, similar to #32.
Ref.

@patrickelectric
Copy link
Member Author

@jaxxzer what you think about that ?

@jaxxzer
Copy link
Member

jaxxzer commented Nov 13, 2017

I've done this: https://github.com/bluerobotics/qgroundcontrol/tree/pr-video-text-overlay

It uses a ton of CPU to re-encode the video, so this implementation at least is not practical for most laptops.

@patrickelectric
Copy link
Member Author

76e9baa CPU problem solved, I'll send a PR ASAP :)

@jaxxzer
Copy link
Member

jaxxzer commented Dec 17, 2018

There was still some issue with this, the encoder is required to embed the text in the video, which required the high cpu use.

@Williangalvani
Copy link
Member

Note: We decided subtitles (advanced substation alpha subtitles) are the best approach to overlay data on video without re-encoding and still allowing to turn the overlay on/off.
The discussion is now taking place here

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants