Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests
@ 2023-03-24  0:25 Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix Michael Niedermayer
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

the IDWT data type mismatched current code

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/tests/snowenc.c | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/libavcodec/tests/snowenc.c b/libavcodec/tests/snowenc.c
index e423ab0541..8064309144 100644
--- a/libavcodec/tests/snowenc.c
+++ b/libavcodec/tests/snowenc.c
@@ -31,6 +31,7 @@ int main(void){
 #define width  256
 #define height 256
     int buffer[2][width*height];
+    short obuffer[width*height];
     SnowContext s;
     int i;
     AVLFG prng;
@@ -49,24 +50,28 @@ int main(void){
 
     printf("testing 5/3 DWT\n");
     for(i=0; i<width*height; i++)
-        buffer[0][i] = buffer[1][i] = av_lfg_get(&prng) % 54321 - 12345;
+        buffer[0][i] = buffer[1][i] = av_lfg_get(&prng) % 19000 - 9000;
 
     ff_spatial_dwt(buffer[0], s.temp_dwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
-    ff_spatial_idwt((IDWTELEM*)buffer[0], s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
+    for(i=0; i<width*height; i++)
+        obuffer[i] = buffer[0][i];
+    ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
 
     for(i=0; i<width*height; i++)
-        if(buffer[0][i]!= buffer[1][i]) printf("fsck: %6d %12d %7d\n",i, buffer[0][i], buffer[1][i]);
+        if(buffer[1][i]!= obuffer[i]) printf("fsck: %4dx%4dx %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
 
     printf("testing 9/7 DWT\n");
     s.spatial_decomposition_type=0;
     for(i=0; i<width*height; i++)
-        buffer[0][i] = buffer[1][i] = av_lfg_get(&prng) % 54321 - 12345;
+        buffer[0][i] = buffer[1][i] = av_lfg_get(&prng) % 11000 - 5000;
 
     ff_spatial_dwt(buffer[0], s.temp_dwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
-    ff_spatial_idwt((IDWTELEM*)buffer[0], s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
+    for(i=0; i<width*height; i++)
+        obuffer[i] = buffer[0][i];
+    ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
 
     for(i=0; i<width*height; i++)
-        if(FFABS(buffer[0][i] - buffer[1][i])>20) printf("fsck: %6d %12d %7d\n",i, buffer[0][i], buffer[1][i]);
+        if(FFABS(buffer[1][i] - obuffer[i])>20) printf("fsck: %4dx%4d %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
 
     {
     int level, orientation, x, y;
@@ -87,12 +92,12 @@ int main(void){
                 if(orientation&1) buf+=w;
                 if(orientation>1) buf+=stride>>1;
 
-                memset(buffer[0], 0, sizeof(int)*width*height);
+                memset(obuffer, 0, sizeof(short)*width*height);
                 buf[w/2 + h/2*stride]= 256*256;
-                ff_spatial_idwt((IDWTELEM*)buffer[0], s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
+                ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
                 for(y=0; y<height; y++){
                     for(x=0; x<width; x++){
-                        int64_t d= buffer[0][x + y*width];
+                        int64_t d= obuffer[x + y*width];
                         error += d*d;
                         if(FFABS(width/2-x)<9 && FFABS(height/2-y)<9 && level==2) printf("%8"PRId64" ", d);
                     }
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
@ 2023-03-24  0:25 ` Michael Niedermayer
  2023-03-24  8:50   ` Anton Khirnov
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation Michael Niedermayer
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 tools/enc_recon_frame_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/enc_recon_frame_test.c b/tools/enc_recon_frame_test.c
index 8f09ded47e..1ea2293ee7 100644
--- a/tools/enc_recon_frame_test.c
+++ b/tools/enc_recon_frame_test.c
@@ -83,7 +83,7 @@ static int frame_hash(FrameChecksum **pc, size_t *nb_c, int64_t ts,
         int linesize = av_image_get_linesize(frame->format, frame->width, p);
         uint32_t checksum = 0;
 
-        for (int j = 0; j < frame->height >> shift_v; j++) {
+        for (int j = 0; j < frame->height >> (shift_v *!!p); j++) {
             checksum = av_adler32_update(checksum, data, linesize);
             data += frame->linesize[p];
         }
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix Michael Niedermayer
@ 2023-03-24  0:25 ` Michael Niedermayer
  2023-03-25 20:38   ` Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches Michael Niedermayer
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/snowenc.c                   |  8 ++++++--
 tests/ref/seek/vsynth_lena-snow        | 28 +++++++++++++-------------
 tests/ref/vsynth/vsynth1-snow          |  8 ++++----
 tests/ref/vsynth/vsynth1-snow-hpel     |  8 ++++----
 tests/ref/vsynth/vsynth2-snow          |  8 ++++----
 tests/ref/vsynth/vsynth2-snow-hpel     |  8 ++++----
 tests/ref/vsynth/vsynth_lena-snow      |  8 ++++----
 tests/ref/vsynth/vsynth_lena-snow-hpel |  8 ++++----
 8 files changed, 44 insertions(+), 40 deletions(-)

diff --git a/libavcodec/snowenc.c b/libavcodec/snowenc.c
index 5fb5906ed8..4cf7ff1145 100644
--- a/libavcodec/snowenc.c
+++ b/libavcodec/snowenc.c
@@ -1552,10 +1552,10 @@ static void calculate_visual_weight(SnowContext *s, Plane *p){
     int level, orientation, x, y;
 
     for(level=0; level<s->spatial_decomposition_count; level++){
+        int64_t error=0;
         for(orientation=level ? 1 : 0; orientation<4; orientation++){
             SubBand *b= &p->band[level][orientation];
             IDWTELEM *ibuf= b->ibuf;
-            int64_t error=0;
 
             memset(s->spatial_idwt_buffer, 0, sizeof(*s->spatial_idwt_buffer)*width*height);
             ibuf[b->width/2 + b->height/2*b->stride]= 256*16;
@@ -1566,9 +1566,13 @@ static void calculate_visual_weight(SnowContext *s, Plane *p){
                     error += d*d;
                 }
             }
-
+            if (orientation == 2)
+                error /= 2;
             b->qlog= (int)(QROOT * log2(352256.0/sqrt(error)) + 0.5);
+            if (orientation != 1)
+                error = 0;
         }
+        p->band[level][1].qlog = p->band[level][2].qlog;
     }
 }
 
diff --git a/tests/ref/seek/vsynth_lena-snow b/tests/ref/seek/vsynth_lena-snow
index 33d6c27463..b2d2d22cda 100644
--- a/tests/ref/seek/vsynth_lena-snow
+++ b/tests/ref/seek/vsynth_lena-snow
@@ -2,45 +2,45 @@ ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st:-1 flags:0  ts:-1.000000
 ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st:-1 flags:1  ts: 1.894167
-ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39806 size:  3640
+ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39690 size:  3640
 ret: 0         st: 0 flags:0  ts: 0.800000
-ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27442 size:  3494
+ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27382 size:  3493
 ret:-1         st: 0 flags:1  ts:-0.320000
 ret:-1         st:-1 flags:0  ts: 2.576668
 ret: 0         st:-1 flags:1  ts: 1.470835
-ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39806 size:  3640
+ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39690 size:  3640
 ret: 0         st: 0 flags:0  ts: 0.360000
-ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16134 size:  3244
+ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16074 size:  3245
 ret:-1         st: 0 flags:1  ts:-0.760000
 ret:-1         st:-1 flags:0  ts: 2.153336
 ret: 0         st:-1 flags:1  ts: 1.047503
-ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27442 size:  3494
+ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27382 size:  3493
 ret: 0         st: 0 flags:0  ts:-0.040000
 ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st: 0 flags:1  ts: 2.840000
-ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52608 size:  3582
+ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52538 size:  3582
 ret: 0         st:-1 flags:0  ts: 1.730004
-ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52608 size:  3582
+ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52538 size:  3582
 ret: 0         st:-1 flags:1  ts: 0.624171
-ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16134 size:  3244
+ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16074 size:  3245
 ret: 0         st: 0 flags:0  ts:-0.480000
 ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st: 0 flags:1  ts: 2.400000
-ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52608 size:  3582
+ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52538 size:  3582
 ret: 0         st:-1 flags:0  ts: 1.306672
-ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39806 size:  3640
+ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39690 size:  3640
 ret: 0         st:-1 flags:1  ts: 0.200839
 ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st: 0 flags:0  ts:-0.920000
 ret: 0         st: 0 flags:1 dts: 0.000000 pts: 0.000000 pos:   5652 size:  3035
 ret: 0         st: 0 flags:1  ts: 2.000000
-ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52608 size:  3582
+ret: 0         st: 0 flags:1 dts: 1.920000 pts: 1.920000 pos:  52538 size:  3582
 ret: 0         st:-1 flags:0  ts: 0.883340
-ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27442 size:  3494
+ret: 0         st: 0 flags:1 dts: 0.960000 pts: 0.960000 pos:  27382 size:  3493
 ret:-1         st:-1 flags:1  ts:-0.222493
 ret:-1         st: 0 flags:0  ts: 2.680000
 ret: 0         st: 0 flags:1  ts: 1.560000
-ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39806 size:  3640
+ret: 0         st: 0 flags:1 dts: 1.440000 pts: 1.440000 pos:  39690 size:  3640
 ret: 0         st:-1 flags:0  ts: 0.460008
-ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16134 size:  3244
+ret: 0         st: 0 flags:1 dts: 0.480000 pts: 0.480000 pos:  16074 size:  3245
 ret:-1         st:-1 flags:1  ts:-0.645825
diff --git a/tests/ref/vsynth/vsynth1-snow b/tests/ref/vsynth/vsynth1-snow
index f20abd2ee4..b0e3a0bfd7 100644
--- a/tests/ref/vsynth/vsynth1-snow
+++ b/tests/ref/vsynth/vsynth1-snow
@@ -1,4 +1,4 @@
-67c10f8d52fcd1103caa675a1408bf6e *tests/data/fate/vsynth1-snow.avi
-136088 tests/data/fate/vsynth1-snow.avi
-bfc0bcc4bc7b956933aa58acc587018d *tests/data/fate/vsynth1-snow.out.rawvideo
-stddev:   22.77 PSNR: 20.98 MAXDIFF:  175 bytes:  7603200/  7603200
+c4c77a6fb926b89fe6591c398f5cd4db *tests/data/fate/vsynth1-snow.avi
+136160 tests/data/fate/vsynth1-snow.avi
+dcf8b3f62d9c3ae2b2d0fbbacbf83e4e *tests/data/fate/vsynth1-snow.out.rawvideo
+stddev:   22.74 PSNR: 20.99 MAXDIFF:  173 bytes:  7603200/  7603200
diff --git a/tests/ref/vsynth/vsynth1-snow-hpel b/tests/ref/vsynth/vsynth1-snow-hpel
index 39780ad8a2..72b082b2ce 100644
--- a/tests/ref/vsynth/vsynth1-snow-hpel
+++ b/tests/ref/vsynth/vsynth1-snow-hpel
@@ -1,4 +1,4 @@
-e62ae25d5040d04622a965bcb27fdb1e *tests/data/fate/vsynth1-snow-hpel.avi
-138446 tests/data/fate/vsynth1-snow-hpel.avi
-57c914cd150f8fc260b5989ce3e5884c *tests/data/fate/vsynth1-snow-hpel.out.rawvideo
-stddev:   22.74 PSNR: 20.99 MAXDIFF:  172 bytes:  7603200/  7603200
+5c9eb93646eb0e5570d37e9adc9625e4 *tests/data/fate/vsynth1-snow-hpel.avi
+138580 tests/data/fate/vsynth1-snow-hpel.avi
+3382bdde624d8bb4af206a5ac6614605 *tests/data/fate/vsynth1-snow-hpel.out.rawvideo
+stddev:   22.71 PSNR: 21.00 MAXDIFF:  171 bytes:  7603200/  7603200
diff --git a/tests/ref/vsynth/vsynth2-snow b/tests/ref/vsynth/vsynth2-snow
index e9607bb7d0..355f89d5f4 100644
--- a/tests/ref/vsynth/vsynth2-snow
+++ b/tests/ref/vsynth/vsynth2-snow
@@ -1,4 +1,4 @@
-0a41e73ddd2f54936490655b46dad4a3 *tests/data/fate/vsynth2-snow.avi
-72868 tests/data/fate/vsynth2-snow.avi
-34a75f5cf8a71159f1a572d9cedcfef9 *tests/data/fate/vsynth2-snow.out.rawvideo
-stddev:   13.73 PSNR: 25.37 MAXDIFF:  162 bytes:  7603200/  7603200
+5e130d6a48b69348eee7f7c76c5869a3 *tests/data/fate/vsynth2-snow.avi
+72942 tests/data/fate/vsynth2-snow.avi
+9b6cee60e3ec0d1f312a8a25a7878fcc *tests/data/fate/vsynth2-snow.out.rawvideo
+stddev:   13.39 PSNR: 25.59 MAXDIFF:  154 bytes:  7603200/  7603200
diff --git a/tests/ref/vsynth/vsynth2-snow-hpel b/tests/ref/vsynth/vsynth2-snow-hpel
index 66839fd6f6..ec3b5dfad2 100644
--- a/tests/ref/vsynth/vsynth2-snow-hpel
+++ b/tests/ref/vsynth/vsynth2-snow-hpel
@@ -1,4 +1,4 @@
-9bc409e4794ee50691a26c9c836d31a7 *tests/data/fate/vsynth2-snow-hpel.avi
-79728 tests/data/fate/vsynth2-snow-hpel.avi
-2cc64d8171175a1532fd7d3ed3011fbf *tests/data/fate/vsynth2-snow-hpel.out.rawvideo
-stddev:   13.70 PSNR: 25.39 MAXDIFF:  162 bytes:  7603200/  7603200
+8edcf0fd7f066972ff77d5b891ed6dde *tests/data/fate/vsynth2-snow-hpel.avi
+79798 tests/data/fate/vsynth2-snow-hpel.avi
+7e0f2a24feda6fb3e54b85511a28c45f *tests/data/fate/vsynth2-snow-hpel.out.rawvideo
+stddev:   13.35 PSNR: 25.62 MAXDIFF:  157 bytes:  7603200/  7603200
diff --git a/tests/ref/vsynth/vsynth_lena-snow b/tests/ref/vsynth/vsynth_lena-snow
index ec29a78483..582c294531 100644
--- a/tests/ref/vsynth/vsynth_lena-snow
+++ b/tests/ref/vsynth/vsynth_lena-snow
@@ -1,4 +1,4 @@
-8e96f337e8f4ccac7d72ef517e1d2208 *tests/data/fate/vsynth_lena-snow.avi
-57680 tests/data/fate/vsynth_lena-snow.avi
-90963cfd2359d460001c94d94256dc2b *tests/data/fate/vsynth_lena-snow.out.rawvideo
-stddev:   10.48 PSNR: 27.72 MAXDIFF:  119 bytes:  7603200/  7603200
+bf2cf9cacc1d98388798be98872049ee *tests/data/fate/vsynth_lena-snow.avi
+57604 tests/data/fate/vsynth_lena-snow.avi
+707a42eb20195913be55ba8dfadf72fb *tests/data/fate/vsynth_lena-snow.out.rawvideo
+stddev:   10.37 PSNR: 27.81 MAXDIFF:  120 bytes:  7603200/  7603200
diff --git a/tests/ref/vsynth/vsynth_lena-snow-hpel b/tests/ref/vsynth/vsynth_lena-snow-hpel
index 2d6edd8a79..67effebc8a 100644
--- a/tests/ref/vsynth/vsynth_lena-snow-hpel
+++ b/tests/ref/vsynth/vsynth_lena-snow-hpel
@@ -1,4 +1,4 @@
-56b14cb1cbb637536233982e87f7ac3e *tests/data/fate/vsynth_lena-snow-hpel.avi
-61764 tests/data/fate/vsynth_lena-snow-hpel.avi
-244b0266127fa354d8485234b2c388e4 *tests/data/fate/vsynth_lena-snow-hpel.out.rawvideo
-stddev:   10.45 PSNR: 27.74 MAXDIFF:  119 bytes:  7603200/  7603200
+c6ec87a11415a99b1a781f9f5bacb722 *tests/data/fate/vsynth_lena-snow-hpel.avi
+61814 tests/data/fate/vsynth_lena-snow-hpel.avi
+40f330397b7acf6bdbb3ec6d908be451 *tests/data/fate/vsynth_lena-snow-hpel.out.rawvideo
+stddev:   10.34 PSNR: 27.83 MAXDIFF:  118 bytes:  7603200/  7603200
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation Michael Niedermayer
@ 2023-03-24  0:25 ` Michael Niedermayer
  2023-03-25 20:38   ` Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test Michael Niedermayer
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/tests/snowenc.c | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/libavcodec/tests/snowenc.c b/libavcodec/tests/snowenc.c
index 8064309144..19635899d9 100644
--- a/libavcodec/tests/snowenc.c
+++ b/libavcodec/tests/snowenc.c
@@ -37,6 +37,7 @@ int main(void){
     AVLFG prng;
     s.spatial_decomposition_count=6;
     s.spatial_decomposition_type=1;
+    int ret = 0;
 
     s.temp_dwt_buffer  = av_calloc(width, sizeof(*s.temp_dwt_buffer));
     s.temp_idwt_buffer = av_calloc(width, sizeof(*s.temp_idwt_buffer));
@@ -58,7 +59,10 @@ int main(void){
     ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
 
     for(i=0; i<width*height; i++)
-        if(buffer[1][i]!= obuffer[i]) printf("fsck: %4dx%4dx %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
+        if(buffer[1][i]!= obuffer[i]) {
+            printf("fsck: %4dx%4dx %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
+            ret = 1;
+        }
 
     printf("testing 9/7 DWT\n");
     s.spatial_decomposition_type=0;
@@ -71,7 +75,10 @@ int main(void){
     ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
 
     for(i=0; i<width*height; i++)
-        if(FFABS(buffer[1][i] - obuffer[i])>20) printf("fsck: %4dx%4d %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
+        if(FFABS(buffer[1][i] - obuffer[i])>20) {
+            printf("fsck: %4dx%4d %12d %7d\n",i%width, i/width, buffer[1][i], obuffer[i]);
+            ret = 1;
+        }
 
     {
     int level, orientation, x, y;
@@ -137,5 +144,5 @@ int main(void){
         }
 
     }
-    return 0;
+    return ret;
 }
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
                   ` (2 preceding siblings ...)
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches Michael Niedermayer
@ 2023-03-24  0:25 ` Michael Niedermayer
  2023-03-25 20:53   ` Michael Niedermayer
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 6/6] tests: Add fate-snowenc Michael Niedermayer
  2023-03-25 20:37 ` [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
  5 siblings, 1 reply; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

---
 libavcodec/tests/snowenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/tests/snowenc.c b/libavcodec/tests/snowenc.c
index 19635899d9..37198cd4e3 100644
--- a/libavcodec/tests/snowenc.c
+++ b/libavcodec/tests/snowenc.c
@@ -93,14 +93,14 @@ int main(void){
                 int w= width  >> (s.spatial_decomposition_count-level);
                 int h= height >> (s.spatial_decomposition_count-level);
                 int stride= width  << (s.spatial_decomposition_count-level);
-                DWTELEM *buf= buffer[0];
+                IDWTELEM *buf= obuffer;
                 int64_t error=0;
 
                 if(orientation&1) buf+=w;
                 if(orientation>1) buf+=stride>>1;
 
                 memset(obuffer, 0, sizeof(short)*width*height);
-                buf[w/2 + h/2*stride]= 256*256;
+                buf[w/2 + h/2*stride]= 8*256;
                 ff_spatial_idwt(obuffer, s.temp_idwt_buffer, width, height, width, s.spatial_decomposition_type, s.spatial_decomposition_count);
                 for(y=0; y<height; y++){
                     for(x=0; x<width; x++){
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [FFmpeg-devel] [PATCH 6/6] tests: Add fate-snowenc
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
                   ` (3 preceding siblings ...)
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test Michael Niedermayer
@ 2023-03-24  0:25 ` Michael Niedermayer
  2023-03-25 20:37 ` [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
  5 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  0:25 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/tests/snowenc.c |  4 +-
 tests/fate/libavcodec.mak  |  4 ++
 tests/ref/fate/snowenc     | 75 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 81 insertions(+), 2 deletions(-)
 create mode 100644 tests/ref/fate/snowenc

diff --git a/libavcodec/tests/snowenc.c b/libavcodec/tests/snowenc.c
index 37198cd4e3..b3c46a8874 100644
--- a/libavcodec/tests/snowenc.c
+++ b/libavcodec/tests/snowenc.c
@@ -106,7 +106,7 @@ int main(void){
                     for(x=0; x<width; x++){
                         int64_t d= obuffer[x + y*width];
                         error += d*d;
-                        if(FFABS(width/2-x)<9 && FFABS(height/2-y)<9 && level==2) printf("%8"PRId64" ", d);
+                        if(FFABS(width/2-x)<9 && FFABS(height/2-y)<9 && level==2) printf("%8"PRId64, d);
                     }
                     if(FFABS(height/2-y)<9 && level==2) printf("\n");
                 }
@@ -137,7 +137,7 @@ int main(void){
             for(y=0; y<height; y++){
                 for(x=0; x<width; x++){
                     int64_t d= buffer[0][x + y*width];
-                    if(FFABS(width/2-x)<9 && FFABS(height/2-y)<9) printf("%8"PRId64" ", d);
+                    if(FFABS(width/2-x)<9 && FFABS(height/2-y)<9) printf("%8"PRId64, d);
                 }
                 if(FFABS(height/2-y)<9) printf("\n");
             }
diff --git a/tests/fate/libavcodec.mak b/tests/fate/libavcodec.mak
index 8f56fae3a8..1cb483e402 100644
--- a/tests/fate/libavcodec.mak
+++ b/tests/fate/libavcodec.mak
@@ -80,6 +80,10 @@ fate-rangecoder: libavcodec/tests/rangecoder$(EXESUF)
 fate-rangecoder: CMD = run libavcodec/tests/rangecoder$(EXESUF)
 fate-rangecoder: CMP = null
 
+FATE_LIBAVCODEC-$(CONFIG_SNOW_ENCODER) += fate-snowenc
+fate-snowenc: libavcodec/tests/snowenc$(EXESUF)
+fate-snowenc: CMD = run libavcodec/tests/snowenc$(EXESUF)
+
 FATE_LIBAVCODEC-yes += fate-mathops
 fate-mathops: libavcodec/tests/mathops$(EXESUF)
 fate-mathops: CMD = run libavcodec/tests/mathops$(EXESUF)
diff --git a/tests/ref/fate/snowenc b/tests/ref/fate/snowenc
new file mode 100644
index 0000000000..e073ef1469
--- /dev/null
+++ b/tests/ref/fate/snowenc
@@ -0,0 +1,75 @@
+testing 5/3 DWT
+testing 9/7 DWT
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0      -6      -4      19      65    -141      65      19      -4      -6       0       0       0
+       0       0       0       0       0      -5      -3      12      43     -95      43      12      -3      -5       0       0       0
+       0       0       0       0       0      43      29    -113    -408     904    -408    -113      29      43       0       0       0
+       0       0       0       0       0      81      54    -216    -774    1710    -774    -216      54      81       0       0       0
+       0       0       0       0       0      43      29    -113    -408     904    -408    -113      29      43       0       0       0
+       0       0       0       0       0      -5      -3      12      43     -95      43      12      -3      -5       0       0       0
+       0       0       0       0       0      -6      -4      19      65    -141      65      19      -4      -6       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0      -6      -4      43      81      43      -4      -6       0       0       0       0       0
+       0       0       0       0       0      -5      -3      28      54      28      -3      -5       0       0       0       0       0
+       0       0       0       0       0      18      12    -114    -216    -114      12      18       0       0       0       0       0
+       0       0       0       0       0      64      43    -409    -774    -409      43      64       0       0       0       0       0
+       0       0       0       0       0    -143     -95     902    1710     902     -95    -143       0       0       0       0       0
+       0       0       0       0       0      64      43    -409    -774    -409      43      64       0       0       0       0       0
+       0       0       0       0       0      18      12    -114    -216    -114      12      18       0       0       0       0       0
+       0       0       0       0       0      -5      -3      28      54      28      -3      -5       0       0       0       0       0
+       0       0       0       0       0      -6      -4      43      81      43      -4      -6       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       3       2      -6     -24      54     -24      -6       2       3       0       0       0
+       0       0       0       0       0       1       1      -5     -16      36     -16      -5       1       1       0       0       0
+       0       0       0       0       0      -6      -4      19      65    -141      65      19      -4      -6       0       0       0
+       0       0       0       0       0     -24     -16      64     231    -511     231      64     -16     -24       0       0       0
+       0       0       0       0       0      54      36    -143    -511    1127    -511    -143      36      54       0       0       0
+       0       0       0       0       0     -24     -16      64     231    -511     231      64     -16     -24       0       0       0
+       0       0       0       0       0      -6      -4      19      65    -141      65      19      -4      -6       0       0       0
+       0       0       0       0       0       1       1      -5     -16      36     -16      -5       1       1       0       0       0
+       0       0       0       0       0       3       2      -6     -24      54     -24      -6       2       3       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0       0
+static int const visual_weight[][4]={
+  {   17661,   10919,   10911,    6749,},
+  {       0,    5176,    5175,    3105,},
+  {       0,    2585,    2584,    1633,},
+};
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
+   52429   52429   52429   52429   52429   52429   52429   52429 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772 -167772
+       0       0       0       0       0       0       0       0       1       1       1       1       1       1       1       1       1
-- 
2.17.1

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix Michael Niedermayer
@ 2023-03-24  8:50   ` Anton Khirnov
  2023-03-24  9:30     ` Michael Niedermayer
  0 siblings, 1 reply; 12+ messages in thread
From: Anton Khirnov @ 2023-03-24  8:50 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Quoting Michael Niedermayer (2023-03-24 01:25:36)
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  tools/enc_recon_frame_test.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/enc_recon_frame_test.c b/tools/enc_recon_frame_test.c
> index 8f09ded47e..1ea2293ee7 100644
> --- a/tools/enc_recon_frame_test.c
> +++ b/tools/enc_recon_frame_test.c
> @@ -83,7 +83,7 @@ static int frame_hash(FrameChecksum **pc, size_t *nb_c, int64_t ts,
>          int linesize = av_image_get_linesize(frame->format, frame->width, p);
>          uint32_t checksum = 0;
>  
> -        for (int j = 0; j < frame->height >> shift_v; j++) {
> +        for (int j = 0; j < frame->height >> (shift_v *!!p); j++) {

IIUC this won't work for alpha. Might as well make it completely
generic.

I'll send an updated version of the tool, thank you for reminding me.

-- 
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix
  2023-03-24  8:50   ` Anton Khirnov
@ 2023-03-24  9:30     ` Michael Niedermayer
  0 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-24  9:30 UTC (permalink / raw)
  To: FFmpeg development discussions and patches


[-- Attachment #1.1: Type: text/plain, Size: 1383 bytes --]

On Fri, Mar 24, 2023 at 09:50:37AM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2023-03-24 01:25:36)
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >  tools/enc_recon_frame_test.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/tools/enc_recon_frame_test.c b/tools/enc_recon_frame_test.c
> > index 8f09ded47e..1ea2293ee7 100644
> > --- a/tools/enc_recon_frame_test.c
> > +++ b/tools/enc_recon_frame_test.c
> > @@ -83,7 +83,7 @@ static int frame_hash(FrameChecksum **pc, size_t *nb_c, int64_t ts,
> >          int linesize = av_image_get_linesize(frame->format, frame->width, p);
> >          uint32_t checksum = 0;
> >  
> > -        for (int j = 0; j < frame->height >> shift_v; j++) {
> > +        for (int j = 0; j < frame->height >> (shift_v *!!p); j++) {
> 
> IIUC this won't work for alpha. Might as well make it completely
> generic.

yes, i didnt really think much about this, it was more a issue i ran into
and wanted to report. After all the tool isnt in git master yet 

> 
> I'll send an updated version of the tool, thank you for reminding me.

[...]

thx
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests
  2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
                   ` (4 preceding siblings ...)
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 6/6] tests: Add fate-snowenc Michael Niedermayer
@ 2023-03-25 20:37 ` Michael Niedermayer
  5 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-25 20:37 UTC (permalink / raw)
  To: FFmpeg development discussions and patches


[-- Attachment #1.1: Type: text/plain, Size: 508 bytes --]

On Fri, Mar 24, 2023 at 01:25:35AM +0100, Michael Niedermayer wrote:
> the IDWT data type mismatched current code
> 
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/tests/snowenc.c | 23 ++++++++++++++---------
>  1 file changed, 14 insertions(+), 9 deletions(-)

will apply

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is a danger to trust the dream we wish for rather than
the science we have, -- Dr. Kenneth Brown

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation Michael Niedermayer
@ 2023-03-25 20:38   ` Michael Niedermayer
  0 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-25 20:38 UTC (permalink / raw)
  To: FFmpeg development discussions and patches


[-- Attachment #1.1: Type: text/plain, Size: 874 bytes --]

On Fri, Mar 24, 2023 at 01:25:37AM +0100, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/snowenc.c                   |  8 ++++++--
>  tests/ref/seek/vsynth_lena-snow        | 28 +++++++++++++-------------
>  tests/ref/vsynth/vsynth1-snow          |  8 ++++----
>  tests/ref/vsynth/vsynth1-snow-hpel     |  8 ++++----
>  tests/ref/vsynth/vsynth2-snow          |  8 ++++----
>  tests/ref/vsynth/vsynth2-snow-hpel     |  8 ++++----
>  tests/ref/vsynth/vsynth_lena-snow      |  8 ++++----
>  tests/ref/vsynth/vsynth_lena-snow-hpel |  8 ++++----
>  8 files changed, 44 insertions(+), 40 deletions(-)

will apply

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is a danger to trust the dream we wish for rather than
the science we have, -- Dr. Kenneth Brown

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches Michael Niedermayer
@ 2023-03-25 20:38   ` Michael Niedermayer
  0 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-25 20:38 UTC (permalink / raw)
  To: FFmpeg development discussions and patches


[-- Attachment #1.1: Type: text/plain, Size: 411 bytes --]

On Fri, Mar 24, 2023 at 01:25:38AM +0100, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/tests/snowenc.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

will apply

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test
  2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test Michael Niedermayer
@ 2023-03-25 20:53   ` Michael Niedermayer
  0 siblings, 0 replies; 12+ messages in thread
From: Michael Niedermayer @ 2023-03-25 20:53 UTC (permalink / raw)
  To: FFmpeg development discussions and patches


[-- Attachment #1.1: Type: text/plain, Size: 463 bytes --]

On Fri, Mar 24, 2023 at 01:25:39AM +0100, Michael Niedermayer wrote:
> ---
>  libavcodec/tests/snowenc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

will apply

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

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

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

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2023-03-25 20:53 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-24  0:25 [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer
2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 2/6] tools/enc_recon_frame_test: plane 0 height fix Michael Niedermayer
2023-03-24  8:50   ` Anton Khirnov
2023-03-24  9:30     ` Michael Niedermayer
2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 3/6] avcodec/snowenc: Fix visual weight calculation Michael Niedermayer
2023-03-25 20:38   ` Michael Niedermayer
2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 4/6] avcodec/tests/snowenc: return a failure if DWT/IDWT mismatches Michael Niedermayer
2023-03-25 20:38   ` Michael Niedermayer
2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 5/6] avcodec/tests/snowenc: Fix 2nd test Michael Niedermayer
2023-03-25 20:53   ` Michael Niedermayer
2023-03-24  0:25 ` [FFmpeg-devel] [PATCH 6/6] tests: Add fate-snowenc Michael Niedermayer
2023-03-25 20:37 ` [FFmpeg-devel] [PATCH 1/6] avcodec/tests/snowenc: unbreak DWT tests Michael Niedermayer

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git