break;
case VIDEO_FRAME_TYPE_3D:
{
- /* We receive the same frame index twice for 3D; hence we know which
+ /* We should receive the same frame index twice for 3D; hence we know which
frame this one is.
*/
- bool const same = (_last_emitted && _last_emitted.get() == frame);
- Data (ContentVideo (image, frame, same ? EYES_RIGHT : EYES_LEFT, PART_WHOLE));
- _last_emitted = frame;
+ bool const same = (_last_emitted_frame && _last_emitted_frame.get() == frame);
+ if (!same && _last_emitted_eyes && *_last_emitted_eyes == EYES_LEFT) {
+ /* We just got a new frame index but the last frame was left-eye; it looks like
+ this content is not really 3D.
+ */
+ boost::throw_exception (
+ DecodeError(
+ String::compose(
+ _("The content file %1 is set as 3D but does not appear to contain 3D images. Please set it to 2D. "
+ "You can still make a 3D DCP from this content by ticking the 3D option in the DCP video tab."),
+ _content->path(0)
+ )
+ )
+ );
+ }
+ Eyes const eyes = same ? EYES_RIGHT : EYES_LEFT;
+ Data (ContentVideo (image, frame, eyes, PART_WHOLE));
+ _last_emitted_frame = frame;
+ _last_emitted_eyes = eyes;
break;
}
case VIDEO_FRAME_TYPE_3D_ALTERNATE:
VideoDecoder::seek ()
{
_position = ContentTime();
- _last_emitted.reset ();
+ _last_emitted_frame.reset ();
+ _last_emitted_eyes.reset ();
}
/*
- Copyright (C) 2013-2016 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2013-2019 Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
* @ingroup completedcp
*/
-#include <boost/test/unit_test.hpp>
#include "test.h"
#include "lib/film.h"
#include "lib/ratio.h"
+#include "lib/config.h"
#include "lib/dcp_content_type.h"
#include "lib/ffmpeg_content.h"
#include "lib/video_content.h"
+#include "lib/job_manager.h"
+#include "lib/cross.h"
+#include "lib/job.h"
+#include <boost/test/unit_test.hpp>
#include <iostream>
using std::cout;
BOOST_AUTO_TEST_CASE (threed_test4)
{
+ Config::instance()->set_master_encoding_threads (8);
+
shared_ptr<Film> film = new_test_film2 ("threed_test4");
shared_ptr<FFmpegContent> L (new FFmpegContent(private_data / "LEFT_TEST_DCP3D4K.mov"));
film->examine_and_add_content (L);
film->write_metadata ();
BOOST_REQUIRE (!wait_for_jobs ());
+
+ Config::instance()->set_master_encoding_threads (8);
}
BOOST_AUTO_TEST_CASE (threed_test5)
{
+ Config::instance()->set_master_encoding_threads (8);
+
shared_ptr<Film> film = new_test_film2 ("threed_test5");
shared_ptr<FFmpegContent> L (new FFmpegContent(private_data / "boon_telly.mkv"));
film->examine_and_add_content (L);
film->write_metadata ();
BOOST_REQUIRE (!wait_for_jobs ());
+
+ Config::instance()->set_master_encoding_threads (1);
}
BOOST_AUTO_TEST_CASE (threed_test6)
check_dcp ("test/data/threed_test6", film->dir(film->dcp_name()));
}
-/** Check 2D content set as being 3D; this fails with a -114 in some versions */
+/** Check 2D content set as being 3D; this should give an informative error */
BOOST_AUTO_TEST_CASE (threed_test7)
{
shared_ptr<Film> film = new_test_film2 ("threed_test7");
film->make_dcp ();
film->write_metadata ();
- BOOST_REQUIRE (!wait_for_jobs());
+ JobManager* jm = JobManager::instance ();
+ while (jm->work_to_do ()) {
+ while (signal_manager->ui_idle()) {}
+ dcpomatic_sleep (1);
+ }
+
+ BOOST_REQUIRE (jm->errors());
+ shared_ptr<Job> failed;
+ BOOST_FOREACH (shared_ptr<Job> i, jm->_jobs) {
+ if (i->finished_in_error()) {
+ BOOST_REQUIRE (!failed);
+ failed = i;
+ }
+ }
+ BOOST_REQUIRE (failed);
+ BOOST_CHECK_EQUAL (failed->error_summary(), "The content file test/data/red_24.mp4 is set as 3D but does not appear to contain 3D images. Please set it to 2D. You can still make a 3D DCP from this content by ticking the 3D option in the DCP video tab.");
}