Re: [FFmpeg-devel] [PATCH v3] doc/filters: Add double-pass example for loudnorm
On 22-08-2018 04:43 AM, Marvin Scholz wrote: +@example +ffmpeg -i input -af loudnorm=I=-23:TP=-1:measured_I=-9.0:measured_TP=1.5:measured_LRA=9.4:measured_thresh=-19.5:print_format=summary output Since your input LRA is 9.4, and output LRA is 7 (default), the filter will upsample to 192 kHz. You should resample afterwards. Ok, whats the correct way to do that? e.g. ffmpeg -i input -af loudnorm=...,aresample=48000 output Ideally, loudnorm should call a resampler internally, but for now, this is what we can do. Thanks, Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 2/2] libavfilter: Removes stored DNN models. Adds support for native backend model file format in tf backend.
On 22-08-2018 01:32 AM, Pedro Arthur wrote: 2018-08-21 16:40 GMT-03:00 Gyan Doshi : Agreed. Can't the filter upscale the chroma planes internally? That's exactly what the filter were doing before, but for some reason doing it this way is not 'good code'. Well, the rule hasn't been applied strictly. The zoompan filter carries out scaling internally, and that hasn't been changed. Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] Correct opus-in-mp4 pre-skip to be uint16_t versus int16_t.
On 8/21/2018 7:45 PM, Dale Curtis wrote: > This field is a uint16_t, see docs: > http://opus-codec.org/docs/opus_in_isobmff.html#4.3.2 > > Signed-off-by: Dale Curtis > > > 0001-Correct-opus-in-mp4-pre-skip-to-be-uint16_t-versus-i.patch > > > From 7f1588bc92ef4a70025aa140a8e660a36875c89c Mon Sep 17 00:00:00 2001 > From: Dale Curtis > Date: Tue, 21 Aug 2018 15:42:31 -0700 > Subject: [PATCH] Correct opus-in-mp4 pre-skip to be uint16_t versus int16_t. > > This field is a uint16_t, see docs: > http://opus-codec.org/docs/opus_in_isobmff.html#4.3.2 > > Signed-off-by: Dale Curtis > --- > libavformat/mov.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index 1bd7d7e483..02f6c1e0a1 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -6608,7 +6608,7 @@ static int mov_read_dops(MOVContext *c, AVIOContext > *pb, MOVAtom atom) > const int OPUS_SEEK_PREROLL_MS = 80; > AVStream *st; > size_t size; > -int16_t pre_skip; > +uint16_t pre_skip; > > if (c->fc->nb_streams < 1) > return 0; > -- 2.18.0.1017.ga543ac7ca45-goog Should be ok. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] Can support h264 encoder by mediacodec on android?
Hello, Ffmpeg has supported H264 hardware decoding by mediacodec.But it cannot support H264 encoder by mediacodec. Can you add H264 encoder by mediacodec in future? Thank's for help. Regards. flhcd -- 成都东方盛行电子有限责任公司 研发中心 冯良怀 2018年8月22日 ADD：成都高新区天华二路81号天府软件园C区C10号楼9楼 P C：610041 TEL:028-85441107-891\892 HTTP：www.ds.net.cn ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] examples/vaapi_dec_scaling: init export
On Tue, Aug 21, 2018 at 4:45 PM Carl Eugen Hoyos wrote: > > 2018-06-11 13:22 GMT+02:00, Jun Zhao : > > > + * Copyright (c) 2018 Jun Zhao > > + * > > + * VA-API Acceleration API (video decoding/scaling) sample > > + * > > + * This file is part of FFmpeg. > > + * > > + * FFmpeg is free software; you can redistribute it and/or > > + * modify it under the terms of the GNU Lesser General Public > > + * License as published by the Free Software Foundation; either > > + * version 2.1 of the License, or (at your option) any later version. > > This is not an ideal license for doc/examples. > I didn't realize this problem, any other license be suggested? MIT ? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avformat/mov: Allow saio/saiz in clear content.
On Mon, Aug 20, 2018 at 10:56:04AM -0700, Jacob Trimble wrote: > On Wed, Aug 15, 2018 at 2:38 PM Michael Niedermayer > wrote: > > > > [...] > > > > > mov.c |6 ++ > > > 1 file changed, 6 insertions(+) > > > 0e583b4ad11852ce38a2b945644e178b7f13a42f > > > 0001-avformat-mov-Allow-saio-saiz-in-clear-content-v2.patch > > > From 256880aca517f64257eb28342a656867d90307a7 Mon Sep 17 00:00:00 2001 > > > From: Jacob Trimble > > > Date: Tue, 14 Aug 2018 10:18:55 -0700 > > > Subject: [PATCH] avformat/mov: Allow saio/saiz in clear content. > > > > This code is used in saio/saiz/senc. The message only mentions the first > > 2. > > > > senc shouldn't appear in clear content, so it shouldn't change > anything. Changed the commit message to include that. will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Its not that you shouldnt use gotos but rather that you should write readable code and code with gotos often but not always is less readable signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] Avoid undefined behavior when start_time_text is -1<<63
On Mon, Aug 20, 2018 at 01:03:31PM -0700, Fredrik Hubinette wrote: > Ok, let's use cast, new patch attached. > > > On Mon, Jul 30, 2018 at 4:53 PM Michael Niedermayer > wrote: > > > On Mon, Jul 30, 2018 at 01:49:24PM -0700, Fredrik Hubinette wrote: > > > Is casting a negative integer to unsigned defined behavior? > > > > yes > > > > 6.3.1.3 Signed and unsigned integers > > > > 2 Otherwise, if the new type is unsigned, the value is converted by > > repeatedly adding or > > subtracting one more than the maximum value that can be represented in > > the new type > > until the value is in the range of the new type.49) > > > > [...] > > -- > > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > > > I do not agree with what you have to say, but I'll defend to the death your > > right to say it. -- Voltaire > > ___ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > utils.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > bc4877b8167668c81cf978a7e3cecfc412ca8c92 > 0001-avoid-undefined-integer-overflow-behavior.patch > From e8f3a5a4ff9b89b33f4067c7aa735fff9895333e Mon Sep 17 00:00:00 2001 > From: Fredrik Hubinette > Date: Mon, 20 Aug 2018 12:59:32 -0700 > Subject: [PATCH] avoid undefined integer overflow behavior will apply thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows, does not speak. He who speaks, does not know. -- Lao Tsu signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 4/7] Adds gray floating-point pixel formats.
On Mon, Aug 20, 2018 at 11:27:05PM +0300, Sergey Lavrushkin wrote: > 2018-08-18 23:20 GMT+03:00 Michael Niedermayer : > > > On Sat, Aug 18, 2018 at 02:10:21PM +0300, Sergey Lavrushkin wrote: > > > 2018-08-17 23:28 GMT+03:00 Michael Niedermayer : > > > > > > > On Fri, Aug 17, 2018 at 12:46:52AM -0300, James Almer wrote: > > > > > On 8/14/2018 1:23 PM, Michael Niedermayer wrote: > > > > > > On Mon, Aug 13, 2018 at 04:58:42PM +0300, Sergey Lavrushkin wrote: > > > > > >>> > > > > > >>> Just use av_clipf instead of FFMIN/FFMAX. > > > > > >> > > > > > >> > > > > > >> Changed FFMIN/FFMAX to av_clip_uint16 and av_clip_uint8. > > > > > > > > > > > > will apply > > > > > > > > > > > > thanks > > > > > > > > > > This broke fate on some 32bit hosts. Guess float pixfmts shouldn't be > > > > > tested for bitexact output. The gbrpf32 ones aren't, for example. > > > > > http://fate.ffmpeg.org/report.cgi?time=20180816131312&slot= > > > > x86_32-debian-kfreebsd-gcc-4.4-cpuflags-mmx > > > > > > > > h > > > > i remember i had tested this locally on 32bit > > > > can something be slightly adjusted (like an offset or factor) to avoid > > any > > > > values becoming close to 0.5 and rounding differently on platforms ? > > > > > > If not the tests should skip float pixel formats or try the nearest > > > > neighbor scaler > > > > > > > > > > Can it really be the problem with scaler? Do all these failed test use > > > scaling? > > > Is not it the problem, that different platforms can give slightly > > different > > > results for > > > floating-point operations? Does input for the float format is somehow > > > generated > > > for these tests, so the input conversion is tested? Maybe it uses output > > > conversion first? > > > If it is the problem of different floating-point operations results on > > > different platforms, > > > > > maybe it is possible to use precomputed LUT for output conversion, so it > > > > I dont think we should change the "algorithm" to achive "bitexactness" > > we could of course but it feels like the wrong reason to make such a > > change. How its done should be choosen based on what is fast (and to a > > lesser extend clean, simple and maintainable) > > > > > > > > > will give > > > the same results? Or is it possible to modify tests for the float format, > > > so it will > > > check if pixels of the result are just close to some reference. > > > > Its possible to compare to a reference, we do this in some other tests, > > but thats surely more work than just disabling teh specific tests or trying > > to nudge them a little to see if that makes nothing fall too close to n + > > 0.5 > > > > > > > > > > > > Sergey, can you look into this (its your patch) ? (just asking to make > > sure > > > > not eevryone thinks someone else will work on this) > > > > > > > > > > Yes, I can, just need to know, what is possible to do to fix this issue, > > > besides skipping the tests. > > > > most things are possible > > > > Hi, > > I am having trouble reproducing this error. These tests are fine for 32-bit > VMs on > my computers. yes, i had the same problem when i initially tested the code, it locally works on 32bit must be the exact platform or compiler ... > So the only thing I can do is to disable these tests for > these formats. > Otherwise, I need to test other changes somehow. Here is the patch, that > skips > pixfmts tests for these formats. in absence of another solution this should be ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Avoid a single point of failure, be that a person or equipment. signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] swscale : hScale16To19 : limit shift for float(32bits) input
On Mon, Aug 20, 2018 at 03:34:26PM +0200, Martin Vignali wrote: > Updated patch in attach > also fix the conversion when using float gray as input, and 8bpc uint as > output > > Martin patches should be ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality is to try to make unequal things equal. -- Aristotle signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] lavc/extract_extradata_bsf.c: add AVS2
On 8/6/2018 10:42 AM, hwren wrote:
> Signed-off-by: hwren
> ---
> libavcodec/extract_extradata_bsf.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libavcodec/extract_extradata_bsf.c
> b/libavcodec/extract_extradata_bsf.c
> index 6deb663..f37427c 100644
> --- a/libavcodec/extract_extradata_bsf.c
> +++ b/libavcodec/extract_extradata_bsf.c
> @@ -331,6 +331,7 @@ static const struct {
> uint8_t **data, int *size);
> } extract_tab[] = {
> { AV_CODEC_ID_AV1,extract_extradata_av1 },
> +{ AV_CODEC_ID_AVS2, extract_extradata_mpeg4 },
> { AV_CODEC_ID_CAVS, extract_extradata_mpeg4 },
> { AV_CODEC_ID_H264, extract_extradata_h2645 },
> { AV_CODEC_ID_HEVC, extract_extradata_h2645 },
> @@ -397,6 +398,7 @@ static void extract_extradata_close(AVBSFContext *ctx)
>
> static const enum AVCodecID codec_ids[] = {
> AV_CODEC_ID_AV1,
> +AV_CODEC_ID_AVS2,
> AV_CODEC_ID_CAVS,
> AV_CODEC_ID_H264,
> AV_CODEC_ID_HEVC,
>
Should be ok.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] lavc/hevc_ps: fix crop info for monochrome
On Mon, Aug 20, 2018 at 10:19:04AM +0800, Zhao Zhili wrote:
>
>
> On 2018年08月18日 05:33, Michael Niedermayer wrote:
> >On Fri, Aug 17, 2018 at 09:52:57AM +0800, Zhao Zhili wrote:
> >>The values of SubWidthC and SubHeightC are 1 in the ITU-T H.265. The
> >>current code use the value of 2.
> >>---
> >> libavcodec/hevc_ps.c | 16
> >> 1 file changed, 12 insertions(+), 4 deletions(-)
> >>
> >>diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c
> >>index fbd9fbf..b56b078 100644
> >>--- a/libavcodec/hevc_ps.c
> >>+++ b/libavcodec/hevc_ps.c
> >>@@ -70,6 +70,14 @@ static const AVRational vui_sar[] = {
> >> { 2, 1 },
> >> };
> >>+static const unsigned hevc_sub_width_c[] = {
> >uint8_t saves a few bytes
> >
> >more important, the commit message should mention a ticket or test sample
> >also a fate test with a _small_ testsample would be usefull. Obviously
> >the existing tests do not cover this
> >
>
> The bug was found by reading the source code. There is no ticket
> related to the bug. I need some time to download the test suite
> and figure out how it work. Feel free to add the test if anyone
> has a suitable sample.
if theres no test sample, creating one would be a good idea so this
is tested. Because as is it would be a change that completely untested
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] swscale/input : avoid float calc for GrayFloat to Gray16 conv
On Mon, Aug 20, 2018 at 04:41:02PM +0200, Martin Vignali wrote: > Hello, > > Patch in attach modify GrayFloat to Gray16 conversion > using the same method currently use inside exr decoder (no float calc) > > duplicate the float_to_uint func inside swscale_internal > > Martin > input.c| 10 +- > swscale_internal.h | 20 > 2 files changed, 25 insertions(+), 5 deletions(-) > 075a95ff08c3976a0b804422c515e49bea3a529b > 0004-swscale-input-avoid-float-calc-for-grayFloat-to-uint.patch > From 742160d2f161b316ad923b0ea0c60d79193c3357 Mon Sep 17 00:00:00 2001 > From: Martin Vignali > Date: Mon, 20 Aug 2018 16:35:13 +0200 > Subject: [PATCH 4/4] swscale/input : avoid float calc for grayFloat to uint16 > conversion > > reuse the float to uint16 func currently use inside exr decoder > --- > libswscale/input.c| 10 +- > libswscale/swscale_internal.h | 20 > 2 files changed, 25 insertions(+), 5 deletions(-) please provide benchmark, what is the impact on speed from this ? thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB During times of universal deceit, telling the truth becomes a revolutionary act. -- George Orwell signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] lavc/extract_extradata_bsf.c: add AVS2
ping
On 08/06/2018 21:42, hwren wrote:
Signed-off-by: hwren
---
libavcodec/extract_extradata_bsf.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavcodec/extract_extradata_bsf.c
b/libavcodec/extract_extradata_bsf.c
index 6deb663..f37427c 100644
--- a/libavcodec/extract_extradata_bsf.c
+++ b/libavcodec/extract_extradata_bsf.c
@@ -331,6 +331,7 @@ static const struct {
uint8_t **data, int *size);
} extract_tab[] = {
{ AV_CODEC_ID_AV1,extract_extradata_av1 },
+{ AV_CODEC_ID_AVS2, extract_extradata_mpeg4 },
{ AV_CODEC_ID_CAVS, extract_extradata_mpeg4 },
{ AV_CODEC_ID_H264, extract_extradata_h2645 },
{ AV_CODEC_ID_HEVC, extract_extradata_h2645 },
@@ -397,6 +398,7 @@ static void extract_extradata_close(AVBSFContext *ctx)
static const enum AVCodecID codec_ids[] = {
AV_CODEC_ID_AV1,
+AV_CODEC_ID_AVS2,
AV_CODEC_ID_CAVS,
AV_CODEC_ID_H264,
AV_CODEC_ID_HEVC,
--
2.7.4
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH v2] lavf: add raw AVS2 demuxer
ping
On 08/03/2018 19:25, hwren wrote:
Signed-off-by: hwren
---
libavformat/Makefile | 1 +
libavformat/allformats.c | 1 +
libavformat/davs2.c | 71
3 files changed, 73 insertions(+)
create mode 100644 libavformat/davs2.c
diff --git a/libavformat/Makefile b/libavformat/Makefile
index f2f3aab..c4534b8 100644
--- a/libavformat/Makefile
+++ b/libavformat/Makefile
@@ -110,6 +110,7 @@ OBJS-$(CONFIG_AST_DEMUXER) += ast.o astdec.o
OBJS-$(CONFIG_AST_MUXER) += ast.o astenc.o
OBJS-$(CONFIG_AU_DEMUXER)+= au.o pcm.o
OBJS-$(CONFIG_AU_MUXER) += au.o rawenc.o
+OBJS-$(CONFIG_AVS2_DEMUXER) += davs2.o rawdec.o
OBJS-$(CONFIG_AVI_DEMUXER) += avidec.o
OBJS-$(CONFIG_AVI_MUXER) += avienc.o mpegtsenc.o avlanguage.o
rawutils.o
OBJS-$(CONFIG_AVM2_MUXER)+= swfenc.o swf.o
diff --git a/libavformat/allformats.c b/libavformat/allformats.c
index adcc8d9..38eaeea 100644
--- a/libavformat/allformats.c
+++ b/libavformat/allformats.c
@@ -76,6 +76,7 @@ extern AVInputFormat ff_avisynth_demuxer;
extern AVOutputFormat ff_avm2_muxer;
extern AVInputFormat ff_avr_demuxer;
extern AVInputFormat ff_avs_demuxer;
+extern AVInputFormat ff_avs2_demuxer;
extern AVInputFormat ff_bethsoftvid_demuxer;
extern AVInputFormat ff_bfi_demuxer;
extern AVInputFormat ff_bintext_demuxer;
diff --git a/libavformat/davs2.c b/libavformat/davs2.c
new file mode 100644
index 000..73daa69
--- /dev/null
+++ b/libavformat/davs2.c
@@ -0,0 +1,71 @@
+/*
+ * AVS2 video stream probe.
+ *
+ * Copyright (C) 2018 Huiwen Ren,
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include "avformat.h"
+#include "rawdec.h"
+#include "libavcodec/internal.h"
+#include "libavutil/intreadwrite.h"
+
+#define ISSQH(x) ((x) == 0xB0 )
+#define ISEND(x) ((x) == 0xB1 )
+#define ISPIC(x) ((x) == 0xB3 || (x) == 0xB6)
+#define ISUNIT(x) ( ISSQH(x) || ISEND(x) || (x) == 0xB2 || ISPIC(x) || (x) ==
0xB5 || (x) == 0xB7 )
+#define ISAVS2(x) ((x) == 0x20 || (x) == 0x22 || (x) == 0x30 || (x) == 0x32 )
+
+static int avs2_probe(AVProbeData *p)
+{
+if (AV_RB32(p->buf) != 0x1B0){
+return 0;
+}
+uint32_t code= -1, hds=0, pic=0, seq=0;
+uint8_t state=0;
+const uint8_t *ptr = p->buf, *end = p->buf + p->buf_size, *sqb=0;
+
+while (ptr < end) {
+ptr = avpriv_find_start_code(ptr, end, &code);
+state = code & 0xFF;
+if ((code & 0xff00) == 0x100) {
+if (ISUNIT(state)) {
+if (sqb && !hds) {
+hds = ptr - sqb;
+}
+if (ISSQH(state)) {
+if (!ISAVS2(*ptr))
+return 0;
+sqb = ptr;
+seq++;
+} else if (ISPIC(state)) {
+pic++;
+} else if (ISEND(state)) {
+break;
+}
+}
+}
+}
+if (seq && hds >= 21 && pic){
+return AVPROBE_SCORE_EXTENSION + 2; // more than cavs
+}
+
+return 0;
+}
+
+FF_DEF_RAWVIDEO_DEMUXER(avs2, "raw AVS2-P2/IEEE1857.4", avs2_probe,
"avs,avs2", AV_CODEC_ID_AVS2)
--
2.7.4
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] mpeg4video: Add Studio DPCM support
$subj Depends on Michael's patch. I have a FATE sample frame. 0001-mpeg4video-Add-Studio-DPCM-support.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] avformat/dashdec: Fix calc_cur_seg_no if availability_start_time not
> On Aug 21, 2018, at 20:33, jacek jendrzej wrote: > > Hi. > Here is an example: > http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd > > ffplay http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd > ffplay version N-91650-g6972dae5a4 Copyright (c) 2003-2018 the FFmpeg > developers > built with gcc 8.2.0 (Gentoo 8.2.0-r2 p1.2) > configuration: --prefix=/mnt/BIG/ffmpeg/root/ --disable-doc > --disable-stripping --enable-debug=1 --enable-libxml2 --enable-demuxer=dash > --enable-openssl > libavutil 56. 19.100 / 56. 19.100 > libavcodec 58. 24.100 / 58. 24.100 > libavformat58. 17.103 / 58. 17.103 > libavdevice58. 4.101 / 58. 4.101 > libavfilter 7. 26.100 / 7. 26.100 > libswscale 5. 2.100 / 5. 2.100 > libswresample 3. 2.100 / 3. 2.100 > [dash @ 0x7ffa8c000b80] Could not read complete fragment.0B f=0/0 > [dash @ 0x7ffa8c000b80] Could not read complete fragment.0B f=0/0 > Input #0, dash, from ' > http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd': > Duration: N/A, start: 1534854670.016000, bitrate: N/A > Program 0 >Stream #0:0: Video: h264 (High) (avc1 / 0x31637661), yuv420p, 640x360, > 30 fps, 30 tbr, 90k tbn, 60 tbc >Metadata: > variant_bitrate : 30 > id : V300 >Stream #0:1: Audio: aac (LC) (mp4a / 0x6134706D), 48000 Hz, stereo, fltp >Metadata: > variant_bitrate : 48000 > id : A48 > [http @ 0x7ffa8c073fc0] HTTP error 404 Not Found 10KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c074780] HTTP error 404 Not Found6KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c236200] HTTP error 404 Not Found0KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c32d940] HTTP error 404 Not Found0KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c086000] HTTP error 404 Not Found0KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0 > [dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0 > [http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0 > <0001-Fix-calc_cur_seg_no-if-availability_start_time-no.patch>___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel LGTM Thanks Steven ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH v3] doc/filters: Add double-pass example for loudnorm
On 21 Aug 2018, at 6:44, Gyan Doshi wrote: On 21-08-2018 03:18 AM, Marvin Scholz wrote: +@example +ffmpeg -i input -af loudnorm=I=-23:TP=-1:measured_I=-9.0:measured_TP=1.5:measured_LRA=9.4:measured_thresh=-19.5:print_format=summary output Since your input LRA is 9.4, and output LRA is 7 (default), the filter will upsample to 192 kHz. You should resample afterwards. Ok, whats the correct way to do that? If we publish this, it will generate support queries as to why our example produces "silent" files. Thanks, Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avcodec/opus_parser: Handle complete frames flag.
On 8/20/2018 3:35 PM, Jacob Trimble wrote: > I am not entirely sure what this flag is supposed to be, since there > is no documentation where it is defined. But this was suggested by > James Almer as a fix for my encrypted Opus problems and several other > codec parsers do the same thing. It's a flag that lets the parser know what kind of frames it's getting. libavformat will set it if the source is a demuxer that guarantees the propagation of complete frames, like mp4, Matroska and Ogg. Applied, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] Correct opus-in-mp4 pre-skip to be uint16_t versus int16_t.
This field is a uint16_t, see docs: http://opus-codec.org/docs/opus_in_isobmff.html#4.3.2 Signed-off-by: Dale Curtis From 7f1588bc92ef4a70025aa140a8e660a36875c89c Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Tue, 21 Aug 2018 15:42:31 -0700 Subject: [PATCH] Correct opus-in-mp4 pre-skip to be uint16_t versus int16_t. This field is a uint16_t, see docs: http://opus-codec.org/docs/opus_in_isobmff.html#4.3.2 Signed-off-by: Dale Curtis --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 1bd7d7e483..02f6c1e0a1 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -6608,7 +6608,7 @@ static int mov_read_dops(MOVContext *c, AVIOContext *pb, MOVAtom atom) const int OPUS_SEEK_PREROLL_MS = 80; AVStream *st; size_t size; -int16_t pre_skip; +uint16_t pre_skip; if (c->fc->nb_streams < 1) return 0; -- 2.18.0.1017.ga543ac7ca45-goog ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 1/9] configure: [loongson] revert no-expensive-optimizations
On Tue, Aug 21, 2018 at 04:06:05PM +0800, Shiyou Yin wrote: > >-Original Message- > >From: ffmpeg-devel-boun...@ffmpeg.org > >[mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > >Shiyou Yin > >Sent: Thursday, July 12, 2018 8:44 PM > >To: ffmpeg-devel@ffmpeg.org > >Subject: [FFmpeg-devel] [PATCH 1/9] configure: [loongson] revert > >no-expensive-optimizations > > > >The bug in gcc-4.9.x has been fixed in gcc master branch. > >Loongson released gcc-4.9.3-3.fc21.loongson with this patch. > >More bug info see: > >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67736 > >https://gcc.gnu.org/ml/gcc-patches/2012-05/msg00401.html > > > >Change-Id: I780125d4cdee71d40457aaee22126ba0547a2c8f > >Signed-off-by: Shiyou Yin > >--- > > configure | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > >diff --git a/configure b/configure > >index b1a4dcf..17a7ea9 100755 > >--- a/configure > >+++ b/configure > >@@ -4789,13 +4789,13 @@ elif enabled mips; then > > disable mipsdspr2 > > case $cpu in > > loongson3*) > >-cpuflags="-march=loongson3a -mhard-float > >-fno-expensive-optimizations" > >+cpuflags="-march=loongson3a -mhard-float" > > ;; > > loongson2e) > >-cpuflags="-march=loongson2e -mhard-float > >-fno-expensive-optimizations" > >+cpuflags="-march=loongson2e -mhard-float" > > ;; > > loongson2f) > >-cpuflags="-march=loongson2f -mhard-float > >-fno-expensive-optimizations" > >+cpuflags="-march=loongson2f -mhard-float" > > ;; > > esac > > ;; > >-- > >2.1.0 > > > > Hi Michael, could you please help to apply this patch. It has been tested on > loongson platform. shouldnt this test the compiler the user uses ? its version or something ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Take away the freedom of one citizen and you will be jailed, take away the freedom of all citizens and you will be congratulated by your peers in Parliament. signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avformat/avformat.h: Add av_stream_remove_side_data.
On Mon, Aug 20, 2018 at 11:41:32AM -0700, Jacob Trimble wrote: > On Mon, Jul 9, 2018 at 9:57 AM Jacob Trimble wrote: > > > > On Tue, Jul 3, 2018 at 5:59 PM Michael Niedermayer > > wrote: > > > > > > On Tue, Jul 03, 2018 at 12:14:19PM -0700, Jacob Trimble wrote: > > > > On Mon, Jul 2, 2018 at 6:07 PM Michael Niedermayer > > > > wrote: > > > > > > > > > > On Mon, Jun 25, 2018 at 04:03:32PM -0700, Jacob Trimble wrote: > > > > > > Signed-off-by: Jacob Trimble > > > > > > --- > > > > > > libavformat/avformat.h | 8 > > > > > > libavformat/utils.c| 11 +++ > > > > > > 2 files changed, 19 insertions(+) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > diff --git a/libavformat/avformat.h b/libavformat/avformat.h > > > > > > index fdaffa5bf4..434c88837e 100644 > > > > > > --- a/libavformat/avformat.h > > > > > > +++ b/libavformat/avformat.h > > > > > > @@ -2167,6 +2167,14 @@ AVStream > > > > > > *avformat_new_stream(AVFormatContext *s, const AVCodec *c); > > > > > > int av_stream_add_side_data(AVStream *st, enum > > > > > > AVPacketSideDataType type, > > > > > > uint8_t *data, size_t size); > > > > > > > > > > > > +/** > > > > > > + * Removes any existing side data of the given type. > > > > > > + * > > > > > > + * @param st stream > > > > > > + * @param type side information type > > > > > > + */ > > > > > > +void av_stream_remove_side_data(AVStream *st, enum > > > > > > AVPacketSideDataType type); > > > > > > > > > > What would use this and why ? > > > > > The commit message does not explain this > > > > > > > > > > If side data is changing it probably should be put in AVPackets or > > > > > AVFrames > > > > > not the stream. > > > > > > > > > > > > > I am using this to removing the side data that contains the > > > > AVEncryptionInitInfo objects once I handle them. Since an MP4 file > > > > can contain multiple pssh atoms, there can be multiple > > > > AVEncryptionInitInfo structs. To make it easier for me, I want to > > > > remove the side data that contain them once I have handled them. This > > > > means that if the AVStream contains the side data, it is because of > > > > new init info I haven't seen. Since the pssh atoms are more "global" > > > > it makes more sense to put them in the AVStream. > > > > > > I dont fully understand but > > > If you intend to remove things while reading the "header" of a mp4 file > > > these things probably should not be in side data to begin with but be > > > internal to the demuxer. > > > > > > otherwise, after the header or outside the demuxer removal seems a "no go" > > > but i may misunderstand what you intend to do. Please explain if iam > > > totally off here with how i interpret this > > > > > > One simple API good vs. bad test btw should be to consider that theres > > > a demuxer connected to a muxer. > > > If this does not work to produce a equivalent file the API is not good > > > for example if you change side data in the AVStream in the middle between > > > outputing packets i would not expect the muxer to see this and thus not > > > be able to reproduce this in the stored file. > > > > > > Also if you mess with the demxuers side data from outside, not only > > > will this result in undefined behavior it also might be that you still > > > need it when for example seeking back to the start > > > > > > again maybe i totally misunderstand what you intend here > > > > > > > I would expect the muxer to do what I am doing, it would remove the > > side data when it handles the data so it doesn't have to keep a copy > > of all the init data it has seen. > > > > For example, consider converting fragmented MP4 into a different > > fragmented MP4. The pssh atoms can appear inside the fragments, so > > the muxer should see the new pssh atoms and add them to the > > current/next fragment while muxing. The best way I see is to check if > > the side data exists, handle it, and remove the side data. The > > alternative would be to convert the side data to the > > AVEncryptionInitInfo struct at every step, then compare each element > > against a copy the muxer has. This is extremely slow and requires > > storing the init data several different ways. > > > > Another alternative would be to put the side data on the frames, but > > this doesn't seem right either. The init info is "header" data, so it > > seems weird to put it on a random frame, and putting the data on every > > frame would be more duplication and require the muxer/app to compare > > them for every frame to detect new init info. > > > > Does that make sense? Is this something that could be merged? how does this work ? in what you describe IIUC there would be a demuxer connected to a muxer its AVPackets would be passed from one to the other, possibly though a FIFO adding arbitrary delay. now if the demxuer updates its stream side data, what will transfer this to the muxer ? they have seperate contexts. And how will this occ
Re: [FFmpeg-devel] [PATCH] configure: do not add --fsanitize= if coverage is tested
On 8/21/2018 5:54 PM, Michael Niedermayer wrote:
> On Mon, Aug 20, 2018 at 10:55:00AM -0300, James Almer wrote:
>> On 8/19/2018 5:40 PM, Michael Niedermayer wrote:
>>> Found-by: Max Moroz
>>> Signed-off-by: Michael Niedermayer
>>> ---
>>> configure | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/configure b/configure
>>> index 9b5421d5a8..b9c9d0b307 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -3964,7 +3964,7 @@ set >> $logfile
>>>
>>> test -n "$valgrind" && toolchain="valgrind-memcheck"
>>>
>>> -enabled ossfuzz && ! echo $CFLAGS | grep -q -- "-fsanitize=" &&{
>>> +enabled ossfuzz && ! echo $CFLAGS | grep -q -- "-fsanitize=" && ! echo
>>> $CFLAGS | grep -q -- "-fcoverage-mapping" &&{
>>
>> What part of configure adds -fcoverage-mapping? Or is this looking for a
>> user set cflag?
>
> user cflags
>
>
>>
>> The subject should be more specific. This is specific for ossfuzz, not
>> other toolchains that add -fsanitize like asan/usan.
>
> what do you suggest ?
"configure: do not add fsanitize cflags with ossfuzz if coverage is
tested" or something like that. Just make it clear that this is not
directly related to the gcov/llvm-cov/usan/asan toolchains.
>
> thx
>
> [...]
>
>
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] configure: do not add --fsanitize= if coverage is tested
On Mon, Aug 20, 2018 at 10:55:00AM -0300, James Almer wrote:
> On 8/19/2018 5:40 PM, Michael Niedermayer wrote:
> > Found-by: Max Moroz
> > Signed-off-by: Michael Niedermayer
> > ---
> > configure | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/configure b/configure
> > index 9b5421d5a8..b9c9d0b307 100755
> > --- a/configure
> > +++ b/configure
> > @@ -3964,7 +3964,7 @@ set >> $logfile
> >
> > test -n "$valgrind" && toolchain="valgrind-memcheck"
> >
> > -enabled ossfuzz && ! echo $CFLAGS | grep -q -- "-fsanitize=" &&{
> > +enabled ossfuzz && ! echo $CFLAGS | grep -q -- "-fsanitize=" && ! echo
> > $CFLAGS | grep -q -- "-fcoverage-mapping" &&{
>
> What part of configure adds -fcoverage-mapping? Or is this looking for a
> user set cflag?
user cflags
>
> The subject should be more specific. This is specific for ossfuzz, not
> other toolchains that add -fsanitize like asan/usan.
what do you suggest ?
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The smallest minority on earth is the individual. Those who deny
individual rights cannot claim to be defenders of minorities. - Ayn Rand
signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] make work (live) libsrt
Hello,
I get errors when I try to send a live srt stream that the first packet is
too big:
21:30:39.896626/ffmpeg*E: SRT.c: LiveSmoother: payload size: 1504 exceeds
maximum allowed 1316
Here are example commands for server and client:
ffmpeg -re -i ~/Downloads/ToS-4k-1920.mov -vcodec libx264 -g 50 -refs 1 -s
640x360 -b:v 1000k -acodec aac -b:a 64k -flush_packets 0 -f mpegts "srt://
127.0.0.1:?mode=listener"
ffplay srt://127.0.0.1:
A patch that fully solves the issue is:
diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c
index 0f9529d263..156a4776e2 100644
--- a/libavformat/libsrt.c
+++ b/libavformat/libsrt.c
@@ -412,6 +412,8 @@ static int libsrt_open(URLContext *h, const char *uri,
int flags)
return AVERROR_UNKNOWN;
}
+h->max_packet_size = 1316;
+
/* SRT options (srt/srt.h) */
p = strchr(uri, '?');
if (p) {
How would you like this option to be made work in a way that can be
accepted in ffmpeg?
Is there a way to change the max packet size without this patch?
Regards,
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 2/2] libavfilter: Removes stored DNN models. Adds support for native backend model file format in tf backend.
2018-08-21 16:40 GMT-03:00 Gyan Doshi : > On 21-08-2018 10:27 PM, Pedro Arthur wrote: > >>> >>> >>> Something like, >>> >>> ffmpeg -i in.bmp -filter_complex >>> 'format=yuvj444p,extractplanes=y+u+v[y][u][v]; >>> [y]sr=dnn_backend=tensorflow:model=espcn.model[y-sr]; >>> [u]scale=iw*2:-1[u2x];[v]scale=iw*2:-1[v2x]; >>> [y-sr][u2x][v2x]mergeplanes=0x001020:yuvj444p' >>> out.bmp >>> >> I personally think the average user, just wanting to upscale and >> image, shouldn't have to type that much. > > > Agreed. Can't the filter upscale the chroma planes internally? That's exactly what the filter were doing before, but for some reason doing it this way is not 'good code'. I think having the power to define graphs of filters were each filter does a single job is nice and elegant but in this case it is doing a disservice to the user, it shouldn't be a strict rule when usability is compromised. > > > Gyan > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] avformat/dashdec: Fix calc_cur_seg_no if availability_start_time not
Hi.
Here is an example:
http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd
ffplay http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd
ffplay version N-91650-g6972dae5a4 Copyright (c) 2003-2018 the FFmpeg
developers
built with gcc 8.2.0 (Gentoo 8.2.0-r2 p1.2)
configuration: --prefix=/mnt/BIG/ffmpeg/root/ --disable-doc
--disable-stripping --enable-debug=1 --enable-libxml2 --enable-demuxer=dash
--enable-openssl
libavutil 56. 19.100 / 56. 19.100
libavcodec 58. 24.100 / 58. 24.100
libavformat58. 17.103 / 58. 17.103
libavdevice58. 4.101 / 58. 4.101
libavfilter 7. 26.100 / 7. 26.100
libswscale 5. 2.100 / 5. 2.100
libswresample 3. 2.100 / 3. 2.100
[dash @ 0x7ffa8c000b80] Could not read complete fragment.0B f=0/0
[dash @ 0x7ffa8c000b80] Could not read complete fragment.0B f=0/0
Input #0, dash, from '
http://vm2.dashif.org/livesim-dev/testpic_2s/Manifest.mpd':
Duration: N/A, start: 1534854670.016000, bitrate: N/A
Program 0
Stream #0:0: Video: h264 (High) (avc1 / 0x31637661), yuv420p, 640x360,
30 fps, 30 tbr, 90k tbn, 60 tbc
Metadata:
variant_bitrate : 30
id : V300
Stream #0:1: Audio: aac (LC) (mp4a / 0x6134706D), 48000 Hz, stereo, fltp
Metadata:
variant_bitrate : 48000
id : A48
[http @ 0x7ffa8c073fc0] HTTP error 404 Not Found 10KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c074780] HTTP error 404 Not Found6KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c236200] HTTP error 404 Not Found0KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c32d940] HTTP error 404 Not Found0KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c086000] HTTP error 404 Not Found0KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0
[dash @ 0x7ffa8c000b80] Failed to open fragment of playlist 0
[http @ 0x7ffa8c2312c0] HTTP error 404 Not Found0KB sq=0B f=0/0
From 08980973a58afbf441e624f2dcd6b7efb537c97e Mon Sep 17 00:00:00 2001
From: Jacek Jendrzej
Date: Tue, 21 Aug 2018 14:15:42 +0200
Subject: [PATCH] lavf/dashdec: Fix calc_cur_seg_no if availability_start_time not
present
---
libavformat/dashdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
index c6dddeb98f..85fbe6aacc 100644
--- a/libavformat/dashdec.c
+++ b/libavformat/dashdec.c
@@ -1296,7 +1296,7 @@ static int64_t calc_cur_seg_no(AVFormatContext *s, struct representation *pls)
if (pls->presentation_timeoffset) {
num = pls->presentation_timeoffset * pls->fragment_timescale / pls->fragment_duration;
} else if (c->publish_time > 0 && !c->availability_start_time) {
-num = pls->first_seq_no + (((c->publish_time - c->availability_start_time) - c->suggested_presentation_delay) * pls->fragment_timescale) / pls->fragment_duration;
+num = pls->first_seq_no + (((c->publish_time - c->time_shift_buffer_depth + pls->fragment_duration) - c->suggested_presentation_delay) * pls->fragment_timescale) / pls->fragment_duration;
} else {
num = pls->first_seq_no + (((get_current_time_in_sec() - c->availability_start_time) - c->suggested_presentation_delay) * pls->fragment_timescale) / pls->fragment_duration;
}
--
2.18.0
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 2/2] libavfilter: Removes stored DNN models. Adds support for native backend model file format in tf backend.
On 21-08-2018 10:27 PM, Pedro Arthur wrote: Something like, ffmpeg -i in.bmp -filter_complex 'format=yuvj444p,extractplanes=y+u+v[y][u][v]; [y]sr=dnn_backend=tensorflow:model=espcn.model[y-sr]; [u]scale=iw*2:-1[u2x];[v]scale=iw*2:-1[v2x]; [y-sr][u2x][v2x]mergeplanes=0x001020:yuvj444p' out.bmp I personally think the average user, just wanting to upscale and image, shouldn't have to type that much. Agreed. Can't the filter upscale the chroma planes internally? Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 08/12] avformat/mxfenc: fix muxing when audio tracks are longer than video track
On Thu, Jul 19, 2018 at 8:39 AM, Baptiste Coudurier < baptiste.coudur...@gmail.com> wrote: > On Wed, Jul 4, 2018 at 11:35 AM, Baptiste Coudurier < > baptiste.coudur...@gmail.com> wrote: > >> --- >> libavformat/mxfenc.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> > > Will apply. > Applied. -- Baptiste ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] avformat/utils: set AV_PKT_FLAG_KEY for data packets
On Mon, Jul 30, 2018 at 11:16 AM, Baptiste Coudurier < baptiste.coudur...@gmail.com> wrote: > --- > libavformat/utils.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index c9cdd2b470..b15f20dd23 100644 > --- a/libavformat/utils.c > +++ b/libavformat/utils.c > @@ -1405,7 +1405,7 @@ static void compute_pkt_fields(AVFormatContext *s, > AVStream *st, > presentation_delayed, delay, av_ts2str(pkt->pts), > av_ts2str(pkt->dts), av_ts2str(st->cur_dts)); > > /* update flags */ > -if (is_intra_only(st->codecpar->codec_id)) > +if (st->codecpar->codec_type == AVMEDIA_TYPE_DATA || > is_intra_only(st->codecpar->codec_id)) > pkt->flags |= AV_PKT_FLAG_KEY; > #if FF_API_CONVERGENCE_DURATION > FF_DISABLE_DEPRECATION_WARNINGS > Will apply. -- Baptiste ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 06/12] avformat/audiointerleave: pad last audio frame
On Thu, Jul 19, 2018 at 8:37 AM, Baptiste Coudurier < baptiste.coudur...@gmail.com> wrote: > On Wed, Jul 4, 2018 at 11:35 AM, Baptiste Coudurier < > baptiste.coudur...@gmail.com> wrote: > >> --- >> libavformat/audiointerleave.c | 10 +++--- >> tests/ref/lavf/mxf| 4 ++-- >> 2 files changed, 9 insertions(+), 5 deletions(-) >> > > Will apply. > > Applied. -- Baptiste COUDURIER ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 2/2] libavfilter: Removes stored DNN models. Adds support for native backend model file format in tf backend.
2018-08-20 15:52 GMT-03:00 Gyan Doshi : > On 20-08-2018 11:52 PM, Sergey Lavrushkin wrote: > >> >> But there are some issues. First, to use this filter for formats with >> chroma channels I do: >> ffmpeg -i in.bmp -filter_complex 'extractplanes=y+u+v[y][u][v]' -map '[y]' >> y.bmp -map '[u]' u.bmp -map '[v]' v.bmp >> ffmpeg -i y.bmp -vf sr=dnn_backend=tensorflow:model=espcn.model y2.bmp >> ffmpeg -i u.bmp -vf scale=iw*2:-1 u2.bmp >> ffmpeg -i v.bmp -vf scale=iw*2:-1 v2.bmp >> ffmpeg -i y2.bmp -i u2.bmp -i v2.bmp -filter_complex >> 'mergeplanes=0x001020:yuv444p' out.bmp >> Can these commands be merged into one command? > > > Something like, > > ffmpeg -i in.bmp -filter_complex >'format=yuvj444p,extractplanes=y+u+v[y][u][v]; > [y]sr=dnn_backend=tensorflow:model=espcn.model[y-sr]; > [u]scale=iw*2:-1[u2x];[v]scale=iw*2:-1[v2x]; > [y-sr][u2x][v2x]mergeplanes=0x001020:yuvj444p' > out.bmp > I personally think the average user, just wanting to upscale and image, shouldn't have to type that much. For the common use case the filter always depends on extractplanes/mergeplanes. > > Regards, > Gyan > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] dashenc.c - variables not passwed to "media_seg_name"
2018-08-21 13:32 GMT+02:00, Jeyapal, Karthick : > > On 8/21/18 2:36 PM, Ibrahim Tachijian wrote: > We could continue our discussion in that bug-tracking thread, > as this devel forum is usually meant for discussing about patches. Or, even better, on the user mailing list. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] dashenc.c - variables not passwed to "media_seg_name"
On 8/21/18 2:36 PM, Ibrahim Tachijian wrote:
> Hey,
>
> Been trying to use the DASH muxer.
> The DASH muxer has this option,
>
> { "media_seg_name", "DASH-templated name to used for the media segments",
> OFFSET(media_seg_name), AV_OPT_TYPE_STRING, {.str =
> "chunk-stream$RepresentationID$-$Number%05d$.m4s"}, 0, 0, E },
>
> It doesnt seem that ffmpeg is parsing the variables inside media_seg_name
> when passing my own "-media_seg_name".
>
> Can somebody confirm?
-media_seg_name is working for me correctly. Could you please recheck your
ffmpeg command line at your end?
If you still think that there is an issue with "media_seg_name", you could
kindly raise a bug with all relevant details in https://trac.ffmpeg.org/. We
could continue our discussion in that bug-tracking thread, as this devel forum
is usually meant for discussing about patches.
>
>
> *Ibrahim Tachijian*CEO Net Sat AB
> Email: bar...@netsat.se
> _
>
>
> *Net Sat AB*Langelandsgatan 8, 164 43 Kista, Sweden
> Office: +46 (0)8 408 394 53
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] avformat/dashenc: Add CODECS tag to HLS master playlist
From: Karthick Jeyapal --- libavformat/dashenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index ae57fd5..229a56e 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -909,7 +909,8 @@ static int write_manifest(AVFormatContext *s, int final) stream_bitrate += max_audio_bitrate; } get_hls_playlist_name(playlist_file, sizeof(playlist_file), NULL, i); -ff_hls_write_stream_info(st, out, stream_bitrate, playlist_file, agroup, NULL, NULL); +ff_hls_write_stream_info(st, out, stream_bitrate, playlist_file, agroup, + os->codec_str, NULL); } avio_close(out); if (use_rename) -- 2.7.4 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] dashenc.c - variables not passwed to "media_seg_name"
Hey,
Been trying to use the DASH muxer.
The DASH muxer has this option,
{ "media_seg_name", "DASH-templated name to used for the media segments",
OFFSET(media_seg_name), AV_OPT_TYPE_STRING, {.str =
"chunk-stream$RepresentationID$-$Number%05d$.m4s"}, 0, 0, E },
It doesnt seem that ffmpeg is parsing the variables inside media_seg_name
when passing my own "-media_seg_name".
Can somebody confirm?
*Ibrahim Tachijian*CEO Net Sat AB
Email: bar...@netsat.se
_
*Net Sat AB*Langelandsgatan 8, 164 43 Kista, Sweden
Office: +46 (0)8 408 394 53
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 2/2] avcodec: add Brooktree ProSumer Video decoder
2018-08-20 23:13 GMT+02:00, Paul B Mahol :
> +for (j = 0; j < avctx->width; j += 8) {
> +*(u++) = *src++;
> +*(y++) = *src++;
> +*(v++) = *src++;
> +*(y++) = *src++;
> +
> +*(u++) = *src++;
> +*(y++) = *src++;
> +*(v++) = *src++;
> +*(y++) = *src++;
> +
> +*(y++) = *src++;
> +*(y++) = *src++;
> +*(y++) = *src++;
> +*(y++) = *src++;
> +}
Time to finish AV_PIX_FMT_UYYVYY411?
=-)
Please ignore, Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH] examples/vaapi_dec_scaling: init export
2018-06-11 13:22 GMT+02:00, Jun Zhao : > + * Copyright (c) 2018 Jun Zhao > + * > + * VA-API Acceleration API (video decoding/scaling) sample > + * > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. This is not an ideal license for doc/examples. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 1/9] configure: [loongson] revert no-expensive-optimizations
>-Original Message- >From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel-boun...@ffmpeg.org] >On Behalf Of >Shiyou Yin >Sent: Thursday, July 12, 2018 8:44 PM >To: ffmpeg-devel@ffmpeg.org >Subject: [FFmpeg-devel] [PATCH 1/9] configure: [loongson] revert >no-expensive-optimizations > >The bug in gcc-4.9.x has been fixed in gcc master branch. >Loongson released gcc-4.9.3-3.fc21.loongson with this patch. >More bug info see: >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67736 >https://gcc.gnu.org/ml/gcc-patches/2012-05/msg00401.html > >Change-Id: I780125d4cdee71d40457aaee22126ba0547a2c8f >Signed-off-by: Shiyou Yin >--- > configure | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/configure b/configure >index b1a4dcf..17a7ea9 100755 >--- a/configure >+++ b/configure >@@ -4789,13 +4789,13 @@ elif enabled mips; then > disable mipsdspr2 > case $cpu in > loongson3*) >-cpuflags="-march=loongson3a -mhard-float >-fno-expensive-optimizations" >+cpuflags="-march=loongson3a -mhard-float" > ;; > loongson2e) >-cpuflags="-march=loongson2e -mhard-float >-fno-expensive-optimizations" >+cpuflags="-march=loongson2e -mhard-float" > ;; > loongson2f) >-cpuflags="-march=loongson2f -mhard-float >-fno-expensive-optimizations" >+cpuflags="-march=loongson2f -mhard-float" > ;; > esac > ;; >-- >2.1.0 > Hi Michael, could you please help to apply this patch. It has been tested on loongson platform. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
