Fix corrupted image when over-cropping black filler frames.
[dcpomatic.git] / test / image_test.cc
index 2bbe9d14b1e5c789031a8c52a69758287891bc3c..2f66cf41d042919a7ed7374b867419ce5d94ded5 100644 (file)
@@ -24,6 +24,7 @@
  *  @see test/make_black_test.cc, test/pixel_formats_test.cc
  */
 
+#include "lib/compose.hpp"
 #include "lib/image.h"
 #include "lib/ffmpeg_image_proxy.h"
 #include "test.h"
@@ -39,8 +40,8 @@ BOOST_AUTO_TEST_CASE (aligned_image_test)
 {
        Image* s = new Image (AV_PIX_FMT_RGB24, dcp::Size (50, 50), true);
        BOOST_CHECK_EQUAL (s->planes(), 1);
-       /* 160 is 150 aligned to the nearest 32 bytes */
-       BOOST_CHECK_EQUAL (s->stride()[0], 160);
+       /* 192 is 150 aligned to the nearest 64 bytes */
+       BOOST_CHECK_EQUAL (s->stride()[0], 192);
        BOOST_CHECK_EQUAL (s->line_size()[0], 150);
        BOOST_CHECK (s->data()[0]);
        BOOST_CHECK (!s->data()[1]);
@@ -50,7 +51,7 @@ BOOST_AUTO_TEST_CASE (aligned_image_test)
        /* copy constructor */
        Image* t = new Image (*s);
        BOOST_CHECK_EQUAL (t->planes(), 1);
-       BOOST_CHECK_EQUAL (t->stride()[0], 160);
+       BOOST_CHECK_EQUAL (t->stride()[0], 192);
        BOOST_CHECK_EQUAL (t->line_size()[0], 150);
        BOOST_CHECK (t->data()[0]);
        BOOST_CHECK (!t->data()[1]);
@@ -67,7 +68,7 @@ BOOST_AUTO_TEST_CASE (aligned_image_test)
        Image* u = new Image (AV_PIX_FMT_YUV422P, dcp::Size (150, 150), false);
        *u = *s;
        BOOST_CHECK_EQUAL (u->planes(), 1);
-       BOOST_CHECK_EQUAL (u->stride()[0], 160);
+       BOOST_CHECK_EQUAL (u->stride()[0], 192);
        BOOST_CHECK_EQUAL (u->line_size()[0], 150);
        BOOST_CHECK (u->data()[0]);
        BOOST_CHECK (!u->data()[1]);
@@ -267,6 +268,71 @@ BOOST_AUTO_TEST_CASE (crop_scale_window_test)
        check_image("test/data/crop_scale_window_test.png", "build/test/crop_scale_window_test.png");
 }
 
+/** Special cases of Image::crop_scale_window which triggered some valgrind warnings */
+BOOST_AUTO_TEST_CASE (crop_scale_window_test2)
+{
+       shared_ptr<Image> image (new Image(AV_PIX_FMT_XYZ12LE, dcp::Size(2048, 858), true));
+       image->crop_scale_window (Crop(279, 0, 0, 0), dcp::Size(1069, 448), dcp::Size(1069, 578), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, false, false);
+       image->crop_scale_window (Crop(2048, 0, 0, 0), dcp::Size(1069, 448), dcp::Size(1069, 578), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, false, false);
+}
+
+BOOST_AUTO_TEST_CASE (crop_scale_window_test3)
+{
+       shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/player_seek_test_0.png"));
+       shared_ptr<Image> xyz = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, true, false);
+       shared_ptr<Image> cropped = xyz->crop_scale_window(Crop(512, 0, 0, 0), dcp::Size(1486, 1080), dcp::Size(1998, 1080), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, false, false);
+       write_image(cropped, "build/test/crop_scale_window_test3.png", "RGB", MagickCore::CharPixel);
+}
+
+BOOST_AUTO_TEST_CASE (crop_scale_window_test4)
+{
+       shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/player_seek_test_0.png"));
+       shared_ptr<Image> xyz = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, true, false);
+       shared_ptr<Image> cropped = xyz->crop_scale_window(Crop(512, 0, 0, 0), dcp::Size(1486, 1080), dcp::Size(1998, 1080), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_XYZ12LE, false, false);
+       write_image(cropped, "build/test/crop_scale_window_test4.png", "RGB", MagickCore::ShortPixel);
+}
+
+BOOST_AUTO_TEST_CASE (crop_scale_window_test5)
+{
+       shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/player_seek_test_0.png"));
+       shared_ptr<Image> xyz = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_XYZ12LE, true, false);
+       shared_ptr<Image> cropped = xyz->crop_scale_window(Crop(512, 0, 0, 0), dcp::Size(1486, 1080), dcp::Size(1998, 1080), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, false, false);
+       write_image(cropped, "build/test/crop_scale_window_test5.png", "RGB", MagickCore::CharPixel);
+}
+
+BOOST_AUTO_TEST_CASE (crop_scale_window_test6)
+{
+       shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/player_seek_test_0.png"));
+       shared_ptr<Image> xyz = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_XYZ12LE, true, false);
+       shared_ptr<Image> cropped = xyz->crop_scale_window(Crop(512, 0, 0, 0), dcp::Size(1486, 1080), dcp::Size(1998, 1080), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_XYZ12LE, false, false);
+       write_image(cropped, "build/test/crop_scale_window_test6.png", "RGB", MagickCore::ShortPixel);
+}
+
+
+/** Test some small crops with an image that shows up errors in registration of the YUV planes (#1872) */
+BOOST_AUTO_TEST_CASE (crop_scale_window_test7)
+{
+       using namespace boost::filesystem;
+       for (int left_crop = 0; left_crop < 8; ++left_crop) {
+               shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/rgb_grey_testcard.png"));
+               shared_ptr<Image> yuv = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_YUV420P, true, false);
+               int rounded = left_crop - (left_crop % 2);
+               shared_ptr<Image> cropped = yuv->crop_scale_window(
+                       Crop(left_crop, 0, 0, 0),
+                       dcp::Size(1998 - rounded, 1080),
+                       dcp::Size(1998 - rounded, 1080),
+                       dcp::YUV_TO_RGB_REC709,
+                       AV_PIX_FMT_RGB24,
+                       true,
+                       false
+                       );
+               path file = String::compose("crop_scale_window_test7-%1.png", left_crop);
+               write_image(cropped, path("build") / "test" / file, "RGB");
+               check_image(path("test") / "data" / file, path("build") / "test" / file, 10);
+       }
+}
+
+
 BOOST_AUTO_TEST_CASE (as_png_test)
 {
        shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/3d_test/000001.png"));
@@ -278,3 +344,66 @@ BOOST_AUTO_TEST_CASE (as_png_test)
        check_image ("test/data/3d_test/000001.png", "build/test/as_png_rgb.png");
        check_image ("test/data/3d_test/000001.png", "build/test/as_png_bgr.png");
 }
+
+/* Very dumb test to fade black to make sure it stays black */
+static void
+fade_test_format_black (AVPixelFormat f, string name)
+{
+       Image yuv (f, dcp::Size(640, 480), true);
+       yuv.make_black ();
+       yuv.fade (0);
+       string const filename = "fade_test_black_" + name + ".png";
+       yuv.convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGBA, true, false)->as_png().write("build/test/" + filename);
+       check_image ("test/data/" + filename, "build/test/" + filename);
+}
+
+/* Fade red to make sure it stays red */
+static void
+fade_test_format_red (AVPixelFormat f, float amount, string name)
+{
+       shared_ptr<FFmpegImageProxy> proxy(new FFmpegImageProxy("test/data/flat_red.png"));
+       shared_ptr<Image> red = proxy->image().first->convert_pixel_format(dcp::YUV_TO_RGB_REC709, f, true, false);
+       red->fade (amount);
+       string const filename = "fade_test_red_" + name + ".png";
+       red->convert_pixel_format(dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGBA, true, false)->as_png().write("build/test/" + filename);
+       check_image ("test/data/" + filename, "build/test/" + filename);
+}
+
+BOOST_AUTO_TEST_CASE (fade_test)
+{
+       fade_test_format_black (AV_PIX_FMT_YUV420P,   "yuv420p");
+       fade_test_format_black (AV_PIX_FMT_YUV422P10, "yuv422p10");
+       fade_test_format_black (AV_PIX_FMT_RGB24,     "rgb24");
+       fade_test_format_black (AV_PIX_FMT_XYZ12LE,   "xyz12le");
+       fade_test_format_black (AV_PIX_FMT_RGB48LE,   "rgb48le");
+
+       fade_test_format_red   (AV_PIX_FMT_YUV420P,   0,   "yuv420p_0");
+       fade_test_format_red   (AV_PIX_FMT_YUV420P,   0.5, "yuv420p_50");
+       fade_test_format_red   (AV_PIX_FMT_YUV420P,   1,   "yuv420p_100");
+       fade_test_format_red   (AV_PIX_FMT_YUV422P10, 0,   "yuv422p10_0");
+       fade_test_format_red   (AV_PIX_FMT_YUV422P10, 0.5, "yuv422p10_50");
+       fade_test_format_red   (AV_PIX_FMT_YUV422P10, 1,   "yuv422p10_100");
+       fade_test_format_red   (AV_PIX_FMT_RGB24,     0,   "rgb24_0");
+       fade_test_format_red   (AV_PIX_FMT_RGB24,     0.5, "rgb24_50");
+       fade_test_format_red   (AV_PIX_FMT_RGB24,     1,   "rgb24_100");
+       fade_test_format_red   (AV_PIX_FMT_XYZ12LE,   0,   "xyz12le_0");
+       fade_test_format_red   (AV_PIX_FMT_XYZ12LE,   0.5, "xyz12le_50");
+       fade_test_format_red   (AV_PIX_FMT_XYZ12LE,   1,   "xyz12le_100");
+       fade_test_format_red   (AV_PIX_FMT_RGB48LE,   0,   "rgb48le_0");
+       fade_test_format_red   (AV_PIX_FMT_RGB48LE,   0.5, "rgb48le_50");
+       fade_test_format_red   (AV_PIX_FMT_RGB48LE,   1,   "rgb48le_100");
+}
+
+/** Make sure the image isn't corrupted if it is cropped too much.  This can happen when a
+ *  filler 128x128 black frame is emitted from the FFmpegDecoder and the overall crop in either direction
+ *  is greater than 128 pixels.
+ */
+BOOST_AUTO_TEST_CASE (over_crop_test)
+{
+       shared_ptr<Image> image (new Image (AV_PIX_FMT_RGB24, dcp::Size(128, 128), true));
+       image->make_black ();
+       shared_ptr<Image> scaled = image->crop_scale_window (Crop(0, 0, 128, 128), dcp::Size(1323, 565), dcp::Size(1349, 565), dcp::YUV_TO_RGB_REC709, AV_PIX_FMT_RGB24, true, true);
+       string const filename = "over_crop_test.png";
+       write_image (scaled, "build/test/" + filename, "RGB");
+       check_image ("test/data/" + filename, "build/test/" + filename);
+}