Re: [FFmpeg-devel] [PATCH 2/2] doc/examples/muxing: Fix packet duration

[Gonzalo]

El 16/07/16 a las 23:10, Michael Niedermayer escribió:

Set needed fields in st->codec

Fixes Ticket5684

Signed-off-by: Michael Niedermayer 
---
@@ -434,6 +434,8 @@ static void open_video(AVFormatContext *oc, AVCodec *codec, 
OutputStream *ost, A
  fprintf(stderr, "Could not copy the stream parameters\n");
  exit(1);
  }
+ost->st->codec->ticks_per_frame = c->ticks_per_frame;
+ost->st->codec->time_base   = c->time_base;
  }
  
I can verify it fixes the ticket, but creates warnings of 
AVStream::codec being deprecated.


--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] doc/examples/muxing: Fix packet duration

[Gonzalo]

El 21/07/16 a las 20:02, Michael Niedermayer escribió:

On Sun, Jul 17, 2016 at 04:10:39AM +0200, Michael Niedermayer wrote:

Set needed fields in st->codec

Fixes Ticket5684

Signed-off-by: Michael Niedermayer 

nevcairiel can you comment on this ?
is this ok or should i add teh fields to codecpar or do you see some
other solution ?

[...]
Ping.  Can you CC nevcairiel, as he may not read this?  The problem 
right now is that ffmpeg has an api that cannot work without deprecated 
warnings.


--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

[Gonzalo]

El 08/03/16 a las 17:14, Michael Niedermayer escribió:

diff --git a/libavutil/frame.c b/libavutil/frame.c
index 5607206..dde32b0 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -375,6 +375,9 @@ int av_frame_ref(AVFrame *dst, const AVFrame *src)
  {
  int i, ret = 0;
  
+av_assert0(dst->width == 0 && dst->height == 0);

+av_assert0(dst->channels == 0);
+
  dst->format = src->format;
  dst->width  = src->width;
  dst->height = src->height;
Why does it matter what dst->width/height is originally?  They are 
overwritten right after.


--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] fps wrong with codecpar

[Gonzalo]

I emailed libav-user, but got no replies.  Thought it might be better to 
post here.


I compiled ffmpeg HEAD and compiled the one example that uses the new 
api (muxing.c).  It compiles and runs fine, but it creates a movie with 
wrong fps according to ffprobe:


Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'test.mov':
  Metadata:
major_brand : qt
minor_version   : 512
compatible_brands: qt
encoder : Lavf57.40.101
  Duration: 00:00:10.03, start: 0.00, bitrate: 289 kb/s
Stream #0:0(eng): Video: h264 (High) (avc1 / 0x31637661), yuv420p, 
352x288, 220 kb/s, 25.10 fps, 25 tbr, 12800 tbn, 50 tbc (default)

Metadata:
(snip)

Notice it says 25.10 fps, instead of 25fps.  When I converted my 
application in a similar way to muxing.c I got weird fps like 25.1603.
When using the old api, all works fine and the fps are okay (but you get 
complaints about st->codec being deprecated).


--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Install config.h?

[Gonzalo]

Hi,

I would like my program to print out its ffmpeg compile options like 
ffplay or ffmpeg do.  However, that info seems to be part of config.h 
which is not installed.  I would like to make a request to install it 
unless there's other way to get that info.

Thank you very much.

Gonzalo Garramuño
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Install config.h?

[Gonzalo]

El 09/10/15 a las 14:18, Ronald S. Bultje escribió:

avconfig.h is the installed counterpart. If you want something from
config.h available in applications, move it to avconfig.h. config.h will
never be installed.
Thanks.  Find attached a patch for configure to have avconfig.h list the 
configuration and license.



diff --git a/configure b/configure
index b572d59..4196088 100755
--- a/configure
+++ b/configure
@@ -6224,6 +6224,8 @@ cat > $TMPH <___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

[Gonzalo]

This was sent in another mail, but may be lost since it did not specify 
a PATCH.


The patch is to get the license and compile flags from any application, 
I add them to the distribution of avconfig.h.


Thanks.
diff --git a/configure b/configure
index b572d59..4196088 100755
--- a/configure
+++ b/configure
@@ -6224,6 +6224,8 @@ cat > $TMPH <___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

[Gonzalo]

El 09/10/15 a las 16:58, Nicolas George escribió:

This information is already available through avcodec_configuration() and
avcodec_license(). Do you really need a static version? With a shared
library, it may not match.


Thank you very much.  That was exactly the functions I was looking for.  
I apologize for the patch.

patch dropped.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/png: read and write stereo3d frame side data information

[Gonzalo]

El 22/10/15 a las 17:50, Kirill Gavrilov escribió:

Use optional sTER chunk defining side-by-side stereo pair
within "Extensions to the PNG 1.2 Specification", version 1.3.0.

Specification:
http://www.libpng.org/pub/png/spec/register/pngext-1.4.0-pdg.html

Note that patch does not add any image resolution checks defined by spec.

Regards,
Kirill

Do you have an example of writing a stereo image through the API?

--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Broken compilation

[Gonzalo]

I just merged master and run into a broken compile:

make: *** No rule to make target «libavcodec/pixblockdsp_template.c», 
needed by «libavcodec/pixblockdsp.o».


Thanks for looking into it.

--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] pngdec failing with an assertion

[Gonzalo]

$ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png
ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg 
developers

  built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~14.04)
  configuration: --enable-gpl --enable-shared --enable-gray 
--enable-runtime-cpudetect --enable-bzlib --enable-libfreetype 
--enable-libmp3lame --enable-libtheora --enable-libvorbis 
--enable-libvpx --enable-libx264 --enable-zlib

  libavutil  55.  2.100 / 55.  2.100
  libavcodec 57.  3.100 / 57.  3.100
  libavformat57.  2.100 / 57.  2.100
  libavdevice57.  0.100 / 57.  0.100
  libavfilter 6.  8.100 /  6.  8.100
  libswscale  4.  0.100 /  4.  0.100
  libswresample   2.  0.100 /  2.  0.100
  libpostproc54.  0.100 / 54.  0.100
Input #0, image2, from '/media/Linux/anims/wildlife/wildlife.%04d.png':
  Duration: 00:00:10.08, start: 0.00, bitrate: N/A
Stream #0:0: Video: png, rgba64be(pc), 1280x720 [SAR 2834:2834 DAR 
16:9], 25 fps, 25 tbr, 25 tbn, 25 tbc

Assertion 0 failed at libavcodec/pngdec.c:666
Abortado (`core' generado)

The same set of files loads fine with a previous ffmpeg version (6 
months ago).

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] pngdec failing with an assertion

[Gonzalo]

El 27/09/15 a las 22:45, Gonzalo escribió:

$ ffplay /media/Linux/anims/wildlife/wildlife.%04d.png
ffplay version N-75563-g235381e Copyright (c) 2003-2015 the FFmpeg 
developers

  built with gcc 4.8 (Ubuntu 4.8.4-2ubuntu1~14.04)
  configuration: --enable-gpl --enable-shared --enable-gray 
--enable-runtime-cpudetect --enable-bzlib --enable-libfreetype 
--enable-libmp3lame --enable-libtheora --enable-libvorbis 
--enable-libvpx --enable-libx264 --enable-zlib

  libavutil  55.  2.100 / 55.  2.100
  libavcodec 57.  3.100 / 57.  3.100
  libavformat57.  2.100 / 57.  2.100
  libavdevice57.  0.100 / 57.  0.100
  libavfilter 6.  8.100 /  6.  8.100
  libswscale  4.  0.100 /  4.  0.100
  libswresample   2.  0.100 /  2.  0.100
  libpostproc54.  0.100 / 54.  0.100
Input #0, image2, from '/media/Linux/anims/wildlife/wildlife.%04d.png':
  Duration: 00:00:10.08, start: 0.00, bitrate: N/A
Stream #0:0: Video: png, rgba64be(pc), 1280x720 [SAR 2834:2834 DAR 
16:9], 25 fps, 25 tbr, 25 tbn, 25 tbc

Assertion 0 failed at libavcodec/pngdec.c:666
Abortado (`core' generado)

The same set of files loads fine with a previous ffmpeg version (6 
months ago).


Also, the individual files load fine.  It is just the sequence that does 
not work.



___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] pngdec failing with an assertion

[Gonzalo]

Please file a ticket at trac.ffmpeg.org with the source PNGs attached to it.

Timothy


I have done so.  The ticket

#4887: defect: PNG sequences crashffplaywith an assertion (new) 



A set of 10 png files as a zip file has been uploaded to:

​http://www.datafilehost.com/d/4abacc22

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavc/pngdec: Reset has_trns on init

[Gonzalo]

El 28/09/15 a las 16:29, Carl Eugen Hoyos escribió:

Carl Eugen Hoyos  ag.or.at> writes:


Attached patch fixes ticket #4887 for me.

Thanks for the quick turnaround to you and Paul B. Mahol.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/exr: add cineon lin2log trc

[Gonzalo Garramuño]

El 06/03/20 a las 16:33, Mark Reid escribió:

You would perhaps be better creating a specialist filter and
implementing it using OCIO (as suggested
https://github.com/AcademySoftwareFoundation/tac/tree/master/gsoc), or
extending the current 3D LUT code to read Cinespace or other 3D LUT
formats that support a pre-shaper and handle the float->integer
conversion that way.


I currently do have a workflow that does something similar to that, but it
slower and involves more steps :)
You are probably better off using oiiotool (part of Sony's OpenImageIO) 
for doing that sort of color corrections on EXRs.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] Added interrupt to av_read_frame() in utils.c. This allows av_read_frame to return after a timeout period.

[Gonzalo Garramuño]

El 21/11/19 a las 06:42, Carl Eugen Hoyos escribió:

Am Mi., 20. Nov. 2019 um 23:42 Uhr schrieb :

From: Gonzalo Garramuño 

I was under the impression that the following patch was written
by Blake Senftner, bsenftner at earthlink.net
Am I wrong?
He wrote the original patch that had the exit interrupt at the beginning 
of the loop.  I wrote the one here where the exit is at the middle of 
the loop.

In any case, I don't know how to send an email for another user with git.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] libavcodec/exr.c: Made channel name comparisons case insensitive.

[Gonzalo Garramuño]

El 19/01/20 a las 19:00, Paul B Mahol escribió:

I doubt this is good.
Why?  I have images created out of Houdini that do not follow the 
uppercase convention used in the exr.c decoder.  The layers cannot be 
read unless the pattern matches layers like: indirect_diffuse.r.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [RFC] BSF list API

[Gonzalo Garramuño]

On 08/06/2016 06:34 p.m., Marton Balint wrote:

Hi All,

Please comment, thanks,

Be careful that : in Windows do not currently parse well if you need to 
use them in a path, like F:/filter/ or Z:/movie.  They are parsed as F 
and /filter/ and Z and /movie/ respectively.


___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]lavc/psd: Support 1bpp images

[Gonzalo Garramuño]

El 10/01/2017 a las 10:21, Carl Eugen Hoyos escribió:

  }
+} else if (avctx->pix_fmt == AV_PIX_FMT_MONOWHITE) {
+ptr = picture->data[0];
+for (y = 0; y < s->height; y++) {
+memcpy(ptr, ptr_data, s->width + 7 >> 3);
+ptr_data += s->width + 7 >> 3;
+ptr += picture->linesize[0];
+}
Just a nitpick.  I would put s->width + 7 >> 3 in a variable outside the 
loop and use it in memcpy and in the ptr_data.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] OpenEXR update and question

[Gonzalo Garramuno]

As you might not know, there's a new version of the OpenEXR library.  
This version introduces a new compressing scheme from Dreamworks 
Animation which is really great.  It is patented but free to use as long 
as you use the openexr library.
I asked the owner of the patent whether it should be possible to 
re-implement the code in ffmpeg as part of exr.c.  The answer has been 
to rely on the openexr C API functions which are added on top of the C++ 
API code.  But it is not clear if this is even possible as there may be 
compiler trouble in Windows.  My guess is that if mingw compiles openexr 
it would link against a mingw ffmpeg just fine, but applications that 
use msvc would not be able to access the openexr functions.  If msvc 
compiles openexr, mingw ffmpeg would not know about them and would fail 
to link.
I am asking here if my reasoning is correct and what would the best 
approach be.  The author of the patent is not being unreasonable as he 
thinks that "as long as the code is used to read and write openexr 
files, all would be fine and an agreement might be found".


___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] OpenEXR update and question

[Gonzalo Garramuno]

On 25/08/14 19:01, Nicolas George wrote:

L'octidi 8 fructidor, an CCXXII, Gonzalo Garramuno a écrit :

As you might not know, there's a new version of the OpenEXR library.
This version introduces a new compressing scheme from Dreamworks
Animation which is really great.  It is patented but free to use as
long as you use the openexr library.

Can you point to the exact wording of the gratis patent licensing?


Here it is.  For more info refer to PATENTS in the openexr distro:

Additional IP Rights Grant (Patents) "DreamWorks Lossy Compression" 
means the
copyrightable works distributed by DreamWorks Animation as part of the 
OpenEXR
Project. Within the OpenEXR project, DreamWorks Animation hereby grants 
to you

a perpetual, worldwide, non-exclusive, no-charge, royalty-free, irrevocable
(except as stated in this section) patent license to make, have made, use,
import, sell products which incorporate this IP as part of the OpenEXR
standard, transfer, and otherwise run, modify and propagate the contents of
this implementation of DreamWorks Lossy Compression within the OpenEXR 
standard,
where such license applies only to those patent claims, both currently 
owned by

DreamWorks Animation and acquired in the future, licensable by DreamWorks
Animation that are necessarily infringed by this implementation of 
DreamWorks

Lossy Compression. This grant does not include use of DreamWorks Lossy
Compression outside of the OpenEXR standard. This grant does not include 
claims
that would be infringed only as a consequence of further modification of 
this
implementation. If you or your agent or exclusive licensee institute or 
order
or agree to the institution of patent litigation against any entity 
(including
a cross-claim or counterclaim in a lawsuit) alleging that this 
implementation

of DreamWorks Lossy Compression or any code incorporated within this
implementation of DreamWorks Lossy Compression constitutes direct or
contributory patent infringement, or inducement of patent infringement, 
then

any patent rights granted to you under this License for this implementation
of DreamWorks Lossy Compression shall terminate as of the date such 
litigation

is filed.

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] OpenEXR update and question

[Gonzalo Garramuno]

On 27/08/14 11:38, Carl Eugen Hoyos wrote:

Gonzalo Garramuno  gmail.com> writes:


I am asking here if my reasoning is correct and
what would the best approach be.

Please provide a failing sample.

I've uploaded the sample Tree image compressed with dwaa.

http://www.datafilehost.com/d/2bb62649

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] RTMPS to FFM

[Gonzalo Martinez]

Hi I have a client written in C++ that use a Hardware Encoder called Cedar
and Stream the Video and Audio to my RTMPS server but now I need to change
that to Stream locally to a ffserver.

The problem that I have when modify the URL from rtmps to
http://localhost:8090/feed1.ffm is an error in DTS. Specially this.

I/HipcamCameraJNI(  786): sending video pts=880919, dts=9223372036854775808
I/ffmpeg  (  786): Application provided invalid, non monotonically
increasing dts to muxer in stream 0: 800821 >= 800821
E/HipcamCameraJNI(  786): Error while writing video frame: Invalid argument

I need to contract someone to help with this bug, I have an environment
with the build and the code and you can test with me in the camera that is
android based.

Please let me know if someone can take this job or if you need more details
about the error.

Thanks


-- 
Gonzalo Martinez
*blog.deploshark.com <http://blog.deploshark.com>*
www.deploshark.com
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Converting build to CMake

[Gonzalo Garramuño]

On 28/8/22 20:17, Jaime Rios wrote:

Sorry if this has been asked before in the past.

I am wondering if there has been any conversation around changing the
current build setup of ffmpeg to CMake.

The reason I ask is that I am not a fan of having to install MinGW just to
build on Windows, and my experience so far with CMake has been a pleasant
one, when having to build on multiple platforms (Windows/Linux/macOS).

If there has been conversation about moving to CMake, where can I find the
thread?


I would love that to happen myself too.  The main issue is all the 
dependencies that ffmpeg has to take care of. You need to have 
FindLib*.cmake and/or BuldLib*cmake, plus you need to take care of which 
libraries are open source, BSD, GPL, etc. and whether you need to locate 
them with pkgconfig.


It is overall a lot of work, so something would need to get paid to do 
so fulltime.


I have to wonder, as I only have Windows 8.1 so far, doesn't Windows 10 
WSL2 solve the compilation issues by having the lib be compiled on an 
Ubuntu distro?


--
Gonzalo Garramuño
ggarr...@gmail.com

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] Off topic question about format specifiers in the C stdlib

[Gonzalo Garramuno]

On 30/05/15 13:24, Marcus Johnson wrote:

I'm writing my own Video codec, and I'm trying to take multiple frames as
the input with a specifier like %03d, is there a stdlib function for this,
or will I have to write it myself?
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
If your codec works with ffmpeg, ffmpeg already supports the %04d 
notation for frame sequences (exr, png, etc).


--
Gonzalo Garramuño

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Blackmagic Design Intensity Pro 4K - Decklink - Problem to select RGB color space

[Gonzalo Berdeal]

Dear ffmpeg developers,

First of all, congrats for your incredible software.

I am using ffmpeg compiled with option Decklink and linked with Deckling
API to capture hdmi video. I have been using the Intensity Pro with
success. But l have just bought the new Intensity Pro 4K and I can not make
it work (however, it works fine with official desktop software).

I think the problem might be on the need of specifying the color space.
With the official software I have to select "Capture File Format" from
between:

AVI 8-bitYUV
AVI 10-bit YUV
AVI 10-bit RGB
AVI Motion JPEG
DPX 10-bit RGB

but with ffmpeg I can not find any option to specify color space. Option
"bm_v210" allows me to select between default (YUV 8-bit) and uyvy422 (YUV
10-bit), but I would need also "10-bit RGB" option.

Looking inside the source code (libavutil/avconfig.h) I have found
definitions of pix_fmt for RGB444. Could it be possible to add as an extra
option for ffmpeg when using Deckling devices the variable to support more
color spaces? Do you think this could fix the problem with the Intensity
Pro 4K?

Thank you very much in advance.
Best regards.

Gonzalo.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel