Merge branch 'leandromoreira:master' into master

Author: AndreiCherniaev

.github/workflows/docker-image.yml

@@ -0,0 +1,23 @@
+name: Docker Image CI
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+
+jobs:
+
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Run hello example
+      run: make make_hello
+    - name: Run remuxing
+      run: make make_remuxing
+    - name: Run transcoding
+      run: make make_transcoding
+      

0_hello_world.c

@@ -28,6 +28,7 @@ static void save_gray_frame(unsigned char *buf, int wrap, int xsize, int ysize,
 
 int main(int argc, const char *argv[])
 {
+
   if (argc < 2) {
     printf("You need to specify a media file.\n");
     return -1;

3_transcoding.c

@@ -328,17 +328,25 @@ int main(int argc, char *argv[])
   avformat_alloc_output_context2(&encoder->avfc, NULL, NULL, encoder->filename);
   if (!encoder->avfc) {logging("could not allocate memory for output format");return -1;}
 
-  if (!sp.copy_video) {
-    AVRational input_framerate = av_guess_frame_rate(decoder->avfc, decoder->video_avs, NULL);
-    prepare_video_encoder(encoder, decoder->video_avcc, input_framerate, sp);
-  } else {
-    if (prepare_copy(encoder->avfc, &encoder->video_avs, decoder->video_avs->codecpar)) {return -1;}
-  }
+  for (int i = 0; i < decoder->avfc->nb_streams; i++) {
+    if (decoder->avfc->streams[i]->codecpar->codec_type == AVMEDIA_TYPE_VIDEO) 
+    {
+      if (!sp.copy_video) {
+        AVRational input_framerate = av_guess_frame_rate(decoder->avfc, decoder->video_avs, NULL);
+        prepare_video_encoder(encoder, decoder->video_avcc, input_framerate, sp);
+      } else {
+        if (prepare_copy(encoder->avfc, &encoder->video_avs, decoder->video_avs->codecpar)) {return -1;}
+      }
+    }
 
-  if (!sp.copy_audio) {
-    if (prepare_audio_encoder(encoder, decoder->audio_avcc->sample_rate, sp)) {return -1;}
-  } else {
-    if (prepare_copy(encoder->avfc, &encoder->audio_avs, decoder->audio_avs->codecpar)) {return -1;}
+    if (decoder->avfc->streams[i]->codecpar->codec_type == AVMEDIA_TYPE_AUDIO)
+    {
+      if (!sp.copy_audio) {
+        if (prepare_audio_encoder(encoder, decoder->audio_avcc->sample_rate, sp)) {return -1;}
+      } else {
+        if (prepare_copy(encoder->avfc, &encoder->audio_avs, decoder->audio_avs->codecpar)) {return -1;}
+      }
+    } 
   }
 
   if (encoder->avfc->oformat->flags & AVFMT_GLOBALHEADER)
@@ -386,8 +394,11 @@ int main(int argc, char *argv[])
       logging("ignoring all non video or audio packets");
     }
   }
-  // TODO: should I also flush the audio encoder?
-  if (encode_video(decoder, encoder, NULL)) return -1;
+
+  if (!sp.copy_video)
+    if (encode_video(decoder, encoder, NULL)) return -1;
+  if (!sp.copy_audio)
+    if (encode_audio(decoder, encoder, NULL)) return -1;
 
   av_write_trailer(encoder->avfc);
 

Dockerfile

@@ -242,7 +242,7 @@ RUN \
         DIR=/tmp/xvid && \
         mkdir -p ${DIR} && \
         cd ${DIR} && \
-        curl -sLO http://downloads.xvid.org/downloads/xvidcore-${XVID_VERSION}.tar.gz && \
+        curl -sLO https://xvid.com/downloads/xvidcore-${XVID_VERSION}.tar.gz && \
         echo ${XVID_SHA256SUM} | sha256sum --check && \
         tar -zx -f xvidcore-${XVID_VERSION}.tar.gz && \
         cd xvidcore/build/generic && \

Makefile

@@ -11,31 +11,31 @@ fetch_small_bunny_video:
 	./fetch_bbb_video.sh
 
 make_hello: clean
-	docker run -w /files --rm -it  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
+	docker run -i -w /files --rm  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
 	  gcc -L/opt/ffmpeg/lib -I/opt/ffmpeg/include/ /files/0_hello_world.c \
 	  -lavcodec -lavformat -lavfilter -lavdevice -lswresample -lswscale -lavutil \
 	  -o /files/build/hello
 
 run_hello: make_hello
-	docker run -w /files --rm -it -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/hello /files/small_bunny_1080p_60fps.mp4
+	docker run -i -w /files --rm  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/hello /files/small_bunny_1080p_60fps.mp4
 
 make_remuxing: clean
-	docker run -w /files --rm -it  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
+	docker run -i -w /files --rm  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
 	  gcc -L/opt/ffmpeg/lib -I/opt/ffmpeg/include/ /files/2_remuxing.c \
 	  -lavcodec -lavformat -lavfilter -lavdevice -lswresample -lswscale -lavutil \
 	  -o /files/build/remuxing
 
 run_remuxing_ts: make_remuxing
-	docker run -w /files --rm -it -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/remuxing /files/small_bunny_1080p_60fps.mp4 /files/remuxed_small_bunny_1080p_60fps.ts
+	docker run -i -w /files --rm -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/remuxing /files/small_bunny_1080p_60fps.mp4 /files/remuxed_small_bunny_1080p_60fps.ts
 
 run_remuxing_fragmented_mp4: make_remuxing
-	docker run -w /files --rm -it -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/remuxing /files/small_bunny_1080p_60fps.mp4 /files/fragmented_small_bunny_1080p_60fps.mp4 fragmented
+	docker run -i -w /files --rm -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 /files/build/remuxing /files/small_bunny_1080p_60fps.mp4 /files/fragmented_small_bunny_1080p_60fps.mp4 fragmented
 
 make_transcoding: clean
-	docker run -w /files --rm -it  -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
+	docker run -i -w /files --rm -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 \
 	  gcc -g -Wall -L/opt/ffmpeg/lib -I/opt/ffmpeg/include/ /files/3_transcoding.c /files/video_debugging.c \
 	  -lavcodec -lavformat -lavfilter -lavdevice -lswresample -lswscale -lavutil \
 	  -o /files/build/3_transcoding
 
 run_transcoding: make_transcoding
-	docker run -w /files --rm -it -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 ./build/3_transcoding /files/small_bunny_1080p_60fps.mp4 /files/bunny_1s_gop.mp4
+	docker run -i -w /files --rm -v `pwd`:/files leandromoreira/ffmpeg-devel:4.4 ./build/3_transcoding /files/small_bunny_1080p_60fps.mp4 /files/bunny_1s_gop.mp4

README-es.md

@@ -111,6 +111,12 @@ Acabamos de hacer **remuxing** (remultiplexación) aquí, el cual consiste conve
 
 FFmpeg posee [documentación](https://www.ffmpeg.org/ffmpeg.html) que hace un gran trabajo explicando como funciona.
 
+```bash
+# tambien puedes ver la documentacion usando la linea de comandos
+
+ffmpeg -h full | grep -A 10 -B 10 avoid_negative_ts
+```
+
 Para ser breves, el comando de línea para FFmpeg espera el siguiente formato de argumentos para realizar sus acciones `ffmpeg {1} {2} -i {3} {4} {5}`, donde:
 
 1. Opciones globales
@@ -189,7 +195,7 @@ bunny_1080p_60fps.webm
 
 **¿Por qué?** las personas intentaran ver tu video usando una conexión `2G` (edge) en un smartphone de baja gama o una conexión por `fibra` a Internet en los televisores a 4K, por lo tanto tu deberías ofrecer mas de una presentación para el mismo video a diferente tasa de bits.
 
-**¿Cómo?** produciendo una presentación con una tasa de bits entre 3856K y 2000K.
+**¿Cómo?** produciendo una presentación con una tasa de bits entre 964K y 3856K.
 
 ```bash
 $ ffmpeg \
@@ -961,4 +967,4 @@ Vamos a convertir el stream desde `h264` a `h265`, como se espera de la versión
 
 ```
 
-> Ahora, para ser honesto, esto fue [más difícil de lo que pensé](https://github.com/leandromoreira/ffmpeg-libav-tutorial/pull/54), voy a tener que y ya me he metido dentro de [el código fuente de la linea de comandos FFmpeg](https://github.com/leandromoreira/ffmpeg-libav-tutorial/pull/54#issuecomment-570746749) y probarlo bastante, y también pienso que estoy olvidando algo, ya que cuando tengo que forzar `force-cfr` para que el `h264` funcione, me sigue arrojando algunos mensajes como `warning messages (forced frame type (5) at 80 was changed to frame type (3))`.
+> Ahora, para ser honesto, esto fue [más difícil de lo que pensé](https://github.com/leandromoreira/ffmpeg-libav-tutorial/pull/54), voy a tener que y ya me he metido dentro de [el código fuente de la linea de comandos FFmpeg](https://github.com/leandromoreira/ffmpeg-libav-tutorial/pull/54#issuecomment-570746749) y probarlo bastante, y también pienso que estoy olvidando algo, ya que cuando tengo que forzar `force-cfr` para que el `h264` funcione, me sigue arrojando algunos mensajes como `warning messages (forced frame type (5) at 80 was changed to frame type (3))`.