/*
- Copyright (C) 2014-2018 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2014-2020 Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
#include "ffmpeg_image_proxy.h"
#include "image.h"
#include "config.h"
+#include "frame_interval_checker.h"
#include <dcp/dcp.h>
#include <dcp/cpl.h>
#include <dcp/reel.h>
#include "compose.hpp"
#include "text_content.h"
#include "audio_content.h"
+#include "frame_interval_checker.h"
#include <dcp/subtitle_string.h>
#include <sub/ssa_reader.h>
#include <sub/subtitle.h>
--- /dev/null
+/*
+ Copyright (C) 2020 Carl Hetherington <cth@carlh.net>
+
+ This file is part of DCP-o-matic.
+
+ DCP-o-matic is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ DCP-o-matic is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
+
+*/
+
+#include "frame_interval_checker.h"
+#include <iostream>
+
+int const FrameIntervalChecker::_frames = 16;
+
+void
+FrameIntervalChecker::feed (ContentTime time, double frame_rate)
+{
+ /* The caller isn't meant to feed too much data before
+ calling guess() and destroying the FrameIntervalChecker.
+ */
+ DCPOMATIC_ASSERT (_intervals.size() < _frames);
+
+ if (_last) {
+ _intervals.push_back ((time - *_last).seconds() * frame_rate);
+ }
+
+ _last = time;
+}
+
+FrameIntervalChecker::Guess
+FrameIntervalChecker::guess () const
+{
+ if (_intervals.size() < _frames) {
+ /* How soon can you land?
+ * I can't tell.
+ * You can tell me, I'm a doctor.
+ * Nom I mean I'm just not sure.
+ * Can't you take a guess?
+ * Well, not for another two hours.
+ * You can't take a guess for another two hours?
+ */
+ return AGAIN;
+ }
+
+ int near_1 = 0;
+ BOOST_FOREACH (double i, _intervals) {
+ if (i > 0.5) {
+ ++near_1;
+ }
+ }
+
+ return (near_1 < 3 * _frames / 4) ? PROBABLY_3D : PROBABLY_NOT_3D;
+}
+
--- /dev/null
+/*
+ Copyright (C) 2020 Carl Hetherington <cth@carlh.net>
+
+ This file is part of DCP-o-matic.
+
+ DCP-o-matic is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ DCP-o-matic is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
+
+*/
+
+#include "dcpomatic_time.h"
+#include <boost/optional.hpp>
+#include <boost/noncopyable.hpp>
+#include <vector>
+
+class FrameIntervalChecker : public boost::noncopyable
+{
+public:
+ void feed (ContentTime time, double frame_rate);
+
+ enum Guess
+ {
+ AGAIN,
+ PROBABLY_3D,
+ PROBABLY_NOT_3D
+ };
+
+ Guess guess () const;
+
+private:
+ boost::optional<ContentTime> _last;
+ /** Intervals of last frames, in fractions of a frame; i.e. 1 in here
+ * means the last two frames were one frame interval apart according
+ * to the frame rate passed to ::feed().
+ */
+ std::vector<double> _intervals;
+
+ static int const _frames;
+};
+
#include "film.h"
#include "exceptions.h"
#include "video_content.h"
+#include "frame_interval_checker.h"
#include <boost/filesystem.hpp>
#include <iostream>
/*
- Copyright (C) 2012-2018 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2012-2020 Carl Hetherington <cth@carlh.net>
This file is part of DCP-o-matic.
#include "raw_image_proxy.h"
#include "film.h"
#include "log.h"
+#include "frame_interval_checker.h"
#include "compose.hpp"
#include <boost/foreach.hpp>
#include <iostream>
VideoDecoder::VideoDecoder (Decoder* parent, shared_ptr<const Content> c)
: DecoderPart (parent)
, _content (c)
+ , _frame_interval_checker (new FrameIntervalChecker())
{
}
double const afr = _content->active_video_frame_rate(film);
VideoFrameType const vft = _content->video->frame_type();
+ ContentTime frame_time = ContentTime::from_frames (decoder_frame, afr);
+
+ /* Do some heuristics to try and spot the case where the user sets content to 3D
+ * when it is not. We try to tell this by looking at the differences in time between
+ * the first few frames. Real 3D content should have two frames for each timestamp.
+ */
+ if (_frame_interval_checker) {
+ _frame_interval_checker->feed (frame_time, afr);
+ if (_frame_interval_checker->guess() == FrameIntervalChecker::PROBABLY_NOT_3D && _content->video->frame_type() == VIDEO_FRAME_TYPE_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)
+ )
+ )
+ );
+ }
+
+ if (_frame_interval_checker->guess() != FrameIntervalChecker::AGAIN) {
+ _frame_interval_checker.reset ();
+ }
+ }
+
Frame frame;
if (!_position) {
/* This is the first data we have received since initialisation or seek. Set
frame this one is.
*/
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;
_position = boost::none;
_last_emitted_frame.reset ();
_last_emitted_eyes.reset ();
+ _frame_interval_checker.reset (new FrameIntervalChecker());
}
class ImageProxy;
class Image;
class Log;
+class FrameIntervalChecker;
/** @class VideoDecoder
* @brief Parent for classes which decode video.
boost::optional<Frame> _last_emitted_frame;
boost::optional<Eyes> _last_emitted_eyes;
boost::optional<ContentTime> _position;
+ boost::scoped_ptr<FrameIntervalChecker> _frame_interval_checker;
};
#endif
#include "video_decoder.h"
#include "video_mxf_content.h"
#include "j2k_image_proxy.h"
+#include "frame_interval_checker.h"
#include <dcp/mono_picture_asset.h>
#include <dcp/mono_picture_asset_reader.h>
#include <dcp/stereo_picture_asset.h>
filter.cc
ffmpeg_image_proxy.cc
font.cc
+ frame_interval_checker.cc
frame_rate_change.cc
hints.cc
internet.cc
--- /dev/null
+/*
+ Copyright (C) 2020 Carl Hetherington <cth@carlh.net>
+
+ This file is part of DCP-o-matic.
+
+ DCP-o-matic is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ DCP-o-matic is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with DCP-o-matic. If not, see <http://www.gnu.org/licenses/>.
+
+*/
+
+#include "lib/frame_interval_checker.h"
+#include <boost/test/unit_test.hpp>
+
+/** Test of 2D-ish frame timings */
+BOOST_AUTO_TEST_CASE (frame_interval_checker_test1)
+{
+ FrameIntervalChecker checker;
+ ContentTime t(3888);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4012);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3776);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3779);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4010);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4085);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4085);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4012);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3776);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3779);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4010);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4085);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4085);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::PROBABLY_NOT_3D);
+}
+
+/** Test of 3D-ish frame timings */
+BOOST_AUTO_TEST_CASE (frame_interval_checker_test2)
+{
+ FrameIntervalChecker checker;
+ ContentTime t(3888);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(0);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(0);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3776);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(50);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4010);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(2);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4011);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(0);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4000);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(0);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(3776);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(50);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4010);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(2);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::AGAIN);
+ t += ContentTime(4011);
+ checker.feed (t, 24);
+ BOOST_CHECK (checker.guess() == FrameIntervalChecker::PROBABLY_3D);
+}
+
+
file_log_test.cc
file_naming_test.cc
film_metadata_test.cc
+ frame_interval_checker_test.cc
frame_rate_test.cc
image_content_fade_test.cc
image_filename_sorter_test.cc