Closed Bug 980479 Opened 9 years ago Closed 6 years ago

Improve quality of eideticker movies


(Testing Graveyard :: Eideticker, defect)

Not set


(Not tracked)



(Reporter: wlach, Assigned: wlach)



(1 file)

The quality of the videos we're generating with pointgrey cameras is quite bad. After doing some research, ffmpeg's default encoder is quite bad. Apparently with more recent versions, you can do better, but I don't think it is available by default in Ubuntu 12.04, which is what we're currently using as our "default platform".

I have been experimenting a bit with vpxenc, and have gotten much improved results, especially if you do a two-pass analysis. I initially thought it was much slower than ffmpeg, but this appears not to be the case after some testing. I think we should switch to using it.
This patch improves the quality of the generated videos quite a bit. Unfortunately I found out that the version of png2yuv does not currently work correctly on Ubuntu 12.04. Later versions of this tool (e.g. in 13.10) do work, however.

Since I would like to see better movies sooner than later, I'd propose the following:

1. Apply this patch. Default behaviour is maintained for now, this remains a "secret" feature.
2. Update the machine in London with a custom-built copy of png2yuv. Modify the copy of eideticker-ci to pass --use-vpxenc when calling
2. When Ubuntu 14.04 is released in a month, make that the default eideticker platform and remove the --use-vpxenc flag and the ffmpeg path.
Attachment #8387846 - Flags: review?(dave.hunt)
This might be interesting to :whsu as well.
Good job! Thanks William's sharing.
Have a nice weekend! :)
Comment on attachment 8387846 [details] [diff] [review]
Optionally use vpxenc to encode  movies

Review of attachment 8387846 [details] [diff] [review]:

This looks good to me. I'd prefer to drop the 'use' prefix, but r=me regardless, especially if this is going to be removed and made the default once we make 14.04 a requirement.

::: src/eideticker/eideticker/
@@ +84,4 @@
>          self.add_option("--no-capture", action="store_true",
>                          dest="no_capture",
>                          help="Skip video capture (mainly for debugging)")
> +        self.add_option("--use-vpxenc", action="store_true",

Could we make the command line argument simply --vpxenc?
Attachment #8387846 - Flags: review?(dave.hunt) → review+
Closed: 6 years ago
Resolution: --- → FIXED
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.