property == SubtitleContentProperty::SUBTITLE_Y_OFFSET ||
property == SubtitleContentProperty::SUBTITLE_X_SCALE ||
property == SubtitleContentProperty::SUBTITLE_Y_SCALE ||
+ property == SubtitleContentProperty::FONTS ||
property == VideoContentProperty::VIDEO_CROP ||
property == VideoContentProperty::VIDEO_SCALE ||
property == VideoContentProperty::VIDEO_FRAME_RATE ||
property == VideoContentProperty::VIDEO_FADE_IN ||
- property == VideoContentProperty::VIDEO_FADE_OUT
+ property == VideoContentProperty::VIDEO_FADE_OUT ||
+ property == VideoContentProperty::COLOUR_CONVERSION
) {
Changed (frequent);
true
),
Position<int> (
- rint (_video_container_size.width * i->rectangle.x),
- rint (_video_container_size.height * i->rectangle.y)
+ lrint (_video_container_size.width * i->rectangle.x),
+ lrint (_video_container_size.height * i->rectangle.y)
)
)
);
/* Text subtitles (rendered to an image) */
if (!ps.text.empty ()) {
- list<PositionImage> s = render_subtitles (ps.text, _video_container_size);
+ list<PositionImage> s = render_subtitles (ps.text, ps.fonts, _video_container_size);
copy (s.begin (), s.end (), back_inserter (sub_images));
}
shared_ptr<const VideoContent> vc = dynamic_pointer_cast<const VideoContent> (piece->content);
DCPTime s = t - piece->content->position ();
s = min (piece->content->length_after_trim(), s);
- /* We're returning a frame index here so we need to floor() the conversion since we want to know the frame
- that contains t, I think
+ s = max (DCPTime(), s + DCPTime (piece->content->trim_start(), piece->frc));
+
+ /* It might seem more logical here to convert s to a ContentTime (using the FrameRateChange)
+ then convert that ContentTime to frames at the content's rate. However this fails for
+ situations like content at 29.9978733fps, DCP at 30fps. The accuracy of the Time type is not
+ enough to distinguish between the two with low values of time (e.g. 3200 in Time units).
+
+ Instead we convert the DCPTime using the DCP video rate then account for any skip/repeat.
*/
- return max (ContentTime (), ContentTime (s, piece->frc) + piece->content->trim_start ()).frames_floor (vc->video_frame_rate ());
+ return s.frames_floor (piece->frc.dcp) / piece->frc.factor ();
}
DCPTime
Player::content_video_to_dcp (shared_ptr<const Piece> piece, Frame f) const
{
shared_ptr<const VideoContent> vc = dynamic_pointer_cast<const VideoContent> (piece->content);
- ContentTime const c = ContentTime::from_frames (f, vc->video_frame_rate ()) - piece->content->trim_start ();
- return max (DCPTime (), DCPTime (c, piece->frc) + piece->content->position ());
+ /* See comment in dcp_to_content_video */
+ DCPTime const d = DCPTime::from_frames (f * piece->frc.factor(), piece->frc.dcp) - DCPTime (piece->content->trim_start (), piece->frc);
+ return max (DCPTime (), d + piece->content->position ());
}
Frame
s.set_aspect_adjust (xs / ys);
}
ps.text.push_back (s);
+ ps.add_fonts (subtitle_content->fonts ());
}
}
}