2018-12-22 10:15 GMT+01:00, Jun Zhao <mypopy...@gmail.com>: > The Scalable Video Technology for HEVC Encoder (SVT-HEVC Encoder) is an
> HEVC-compliant encoder library core that achieves excellent density-quality I don't find it ideal that FFmpeg git claims that an Intel encoder does something "excellent" - unrelated to our actual opinion. Please rephrase. > tradeoffs, and is highly optimized for Intel Xeon Scalable Processor and (and while there, also remove / change "highly optimized") > Xeon D processors. Intel open source SVT-HEVC encoder in: > https://github.com/intel/SVT-HEVC. > This wrapper work with SVT-HEVC master branch, more information can get > from https://github.com/intel/SVT-HEVC/blob/new_api/ffmpeg_plugin/. This is irrelevant: If SVT is another example of a library that does not plan some kind of api stability, we should not merge your patch. If api stability is to be expected, the sentence is not necessary. > For SVT-HEVC build, you can switch the branch to master, then run: This should not be part of the commit message here. [...] Please either: Add some quality metrics in the email. Or: Explain that the license (and the fact that the encoder is not terrible) is sufficient reason to merge. Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
