X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Ffilm_viewer.cc;h=c17927ac5bc0de8d483004ed4c41dcd69c9bbb16;hb=a78b741c43830c84bcb4d18e3147746f13a668e5;hp=595fd4720e413cb417761251f60f82dcbff4e8a9;hpb=49e401f651ccf6c409fc84f57aa205f1920ef070;p=dcpomatic.git diff --git a/src/wx/film_viewer.cc b/src/wx/film_viewer.cc index 595fd4720..c17927ac5 100644 --- a/src/wx/film_viewer.cc +++ b/src/wx/film_viewer.cc @@ -1,19 +1,20 @@ /* - Copyright (C) 2012-2014 Carl Hetherington + Copyright (C) 2012-2016 Carl Hetherington - This program is free software; you can redistribute it and/or modify + 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. - This program is distributed in the hope that it will be useful, + 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 this program; if not, write to the Free Software - Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. + along with DCP-o-matic. If not, see . */ @@ -21,24 +22,32 @@ * @brief A wx widget to view a preview of a Film. */ -#include -#include -#include +#include "film_viewer.h" +#include "playhead_to_timecode_dialog.h" +#include "playhead_to_frame_dialog.h" +#include "wx_util.h" #include "lib/film.h" #include "lib/ratio.h" #include "lib/util.h" #include "lib/job_manager.h" #include "lib/image.h" -#include "lib/scaler.h" #include "lib/exceptions.h" #include "lib/examine_content_job.h" #include "lib/filter.h" #include "lib/player.h" -#include "lib/player_video_frame.h" +#include "lib/player_video.h" #include "lib/video_content.h" #include "lib/video_decoder.h" -#include "film_viewer.h" -#include "wx_util.h" +#include "lib/timer.h" +#include "lib/log.h" +#include "lib/config.h" +extern "C" { +#include +} +#include +#include +#include +#include using std::string; using std::pair; @@ -48,33 +57,48 @@ using std::cout; using std::list; using std::bad_alloc; using std::make_pair; +using std::exception; using boost::shared_ptr; using boost::dynamic_pointer_cast; using boost::weak_ptr; -using libdcp::Size; +using boost::optional; +using dcp::Size; FilmViewer::FilmViewer (wxWindow* p) : wxPanel (p) , _panel (new wxPanel (this)) + , _outline_content (new wxCheckBox (this, wxID_ANY, _("Outline content"))) + , _left_eye (new wxRadioButton (this, wxID_ANY, _("Left eye"), wxDefaultPosition, wxDefaultSize, wxRB_GROUP)) + , _right_eye (new wxRadioButton (this, wxID_ANY, _("Right eye"))) + , _jump_to_selected (new wxCheckBox (this, wxID_ANY, _("Jump to selected content"))) , _slider (new wxSlider (this, wxID_ANY, 0, 0, 4096)) , _back_button (new wxButton (this, wxID_ANY, wxT("<"))) , _forward_button (new wxButton (this, wxID_ANY, wxT(">"))) , _frame_number (new wxStaticText (this, wxID_ANY, wxT(""))) , _timecode (new wxStaticText (this, wxID_ANY, wxT(""))) , _play_button (new wxToggleButton (this, wxID_ANY, _("Play"))) - , _got_frame (false) + , _coalesce_player_changes (false) + , _pending_player_change (false) + , _last_seek_accurate (true) { #ifndef __WXOSX__ _panel->SetDoubleBuffered (true); #endif - + _panel->SetBackgroundStyle (wxBG_STYLE_PAINT); - + _v_sizer = new wxBoxSizer (wxVERTICAL); SetSizer (_v_sizer); _v_sizer->Add (_panel, 1, wxEXPAND); + wxBoxSizer* view_options = new wxBoxSizer (wxHORIZONTAL); + view_options->Add (_outline_content, 0, wxRIGHT, DCPOMATIC_SIZER_GAP); + view_options->Add (_left_eye, 0, wxLEFT | wxRIGHT, DCPOMATIC_SIZER_GAP); + view_options->Add (_right_eye, 0, wxLEFT | wxRIGHT, DCPOMATIC_SIZER_GAP); + view_options->Add (_jump_to_selected, 0, wxLEFT | wxRIGHT, DCPOMATIC_SIZER_GAP); + _v_sizer->Add (view_options, 0, wxALL, DCPOMATIC_SIZER_GAP); + wxBoxSizer* h_sizer = new wxBoxSizer (wxHORIZONTAL); wxBoxSizer* time_sizer = new wxBoxSizer (wxVERTICAL); @@ -93,96 +117,153 @@ FilmViewer::FilmViewer (wxWindow* p) _back_button->SetMinSize (wxSize (32, -1)); _forward_button->SetMinSize (wxSize (32, -1)); - _panel->Bind (wxEVT_PAINT, boost::bind (&FilmViewer::paint_panel, this)); - _panel->Bind (wxEVT_SIZE, boost::bind (&FilmViewer::panel_sized, this, _1)); - _slider->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind (&FilmViewer::slider_moved, this)); - _slider->Bind (wxEVT_SCROLL_PAGEUP, boost::bind (&FilmViewer::slider_moved, this)); - _slider->Bind (wxEVT_SCROLL_PAGEDOWN, boost::bind (&FilmViewer::slider_moved, this)); - _play_button->Bind (wxEVT_COMMAND_TOGGLEBUTTON_CLICKED, boost::bind (&FilmViewer::play_clicked, this)); - _timer.Bind (wxEVT_TIMER, boost::bind (&FilmViewer::timer, this)); - _back_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&FilmViewer::back_clicked, this)); - _forward_button->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&FilmViewer::forward_clicked, this)); + _panel->Bind (wxEVT_PAINT, boost::bind (&FilmViewer::paint_panel, this)); + _panel->Bind (wxEVT_SIZE, boost::bind (&FilmViewer::panel_sized, this, _1)); + _outline_content->Bind (wxEVT_CHECKBOX, boost::bind (&FilmViewer::refresh_panel, this)); + _left_eye->Bind (wxEVT_RADIOBUTTON, boost::bind (&FilmViewer::refresh, this)); + _right_eye->Bind (wxEVT_RADIOBUTTON, boost::bind (&FilmViewer::refresh, this)); + _slider->Bind (wxEVT_SCROLL_THUMBTRACK, boost::bind (&FilmViewer::slider_moved, this)); + _slider->Bind (wxEVT_SCROLL_PAGEUP, boost::bind (&FilmViewer::slider_moved, this)); + _slider->Bind (wxEVT_SCROLL_PAGEDOWN, boost::bind (&FilmViewer::slider_moved, this)); + _play_button->Bind (wxEVT_TOGGLEBUTTON, boost::bind (&FilmViewer::play_clicked, this)); + _timer.Bind (wxEVT_TIMER, boost::bind (&FilmViewer::timer, this)); + _back_button->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::back_clicked, this, _1)); + _forward_button->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::forward_clicked, this, _1)); + _frame_number->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::frame_number_clicked, this)); + _timecode->Bind (wxEVT_LEFT_DOWN, boost::bind (&FilmViewer::timecode_clicked, this)); + _jump_to_selected->Bind (wxEVT_CHECKBOX, boost::bind (&FilmViewer::jump_to_selected_clicked, this)); + + _jump_to_selected->SetValue (Config::instance()->jump_to_selected ()); set_film (shared_ptr ()); - + JobManager::instance()->ActiveJobsChanged.connect ( - bind (&FilmViewer::active_jobs_changed, this, _1) + bind (&FilmViewer::active_jobs_changed, this, _2) ); + + setup_sensitivity (); } void -FilmViewer::set_film (shared_ptr f) +FilmViewer::set_film (shared_ptr film) { - if (_film == f) { + if (_film == film) { return; } - _film = f; + _film = film; _frame.reset (); - - _slider->SetValue (0); - set_position_text (0); - + + update_position_slider (); + update_position_label (); + if (!_film) { return; } try { - _player = f->make_player (); + _player.reset (new Player (_film, _film->playlist ())); + _player->set_fast (); } catch (bad_alloc) { error_dialog (this, _("There is not enough free memory to do that.")); _film.reset (); return; } - - _player->disable_audio (); - _player->Video.connect (boost::bind (&FilmViewer::process_video, this, _1, _3)); + + /* Always burn in subtitles, even if content is set not to, otherwise we won't see them + in the preview. + */ + _player->set_always_burn_subtitles (true); + _player->set_ignore_audio (); + _player->set_play_referenced (); + + _film->Changed.connect (boost::bind (&FilmViewer::film_changed, this, _1)); + _player->Changed.connect (boost::bind (&FilmViewer::player_changed, this, _1)); + _player->Video.connect (boost::bind (&FilmViewer::video, this, _1, _2)); calculate_sizes (); - fetch_next_frame (); + refresh (); + + setup_sensitivity (); } void -FilmViewer::fetch_current_frame_again () +FilmViewer::refresh_panel () { - if (!_player) { - return; - } - - /* We could do this with a seek and a fetch_next_frame, but this is - a shortcut to make it quicker. - */ - - _got_frame = false; - if (!_player->repeat_last_video ()) { - fetch_next_frame (); - } - _panel->Refresh (); _panel->Update (); } void -FilmViewer::timer () +FilmViewer::video (shared_ptr pv, DCPTime time) { if (!_player) { return; } - - fetch_next_frame (); - Time const len = _film->length (); - - if (len) { - int const new_slider_position = 4096 * _player->video_position() / len; - if (new_slider_position != _slider->GetValue()) { - _slider->SetValue (new_slider_position); + if (_film->three_d ()) { + if ((_left_eye->GetValue() && pv->eyes() == EYES_RIGHT) || (_right_eye->GetValue() && pv->eyes() == EYES_LEFT)) { + return; } } + + /* In an ideal world, what we would do here is: + * + * 1. convert to XYZ exactly as we do in the DCP creation path. + * 2. convert back to RGB for the preview display, compensating + * for the monitor etc. etc. + * + * but this is inefficient if the source is RGB. Since we don't + * (currently) care too much about the precise accuracy of the preview's + * colour mapping (and we care more about its speed) we try to short- + * circuit this "ideal" situation in some cases. + * + * The content's specified colour conversion indicates the colourspace + * which the content is in (according to the user). + * + * PlayerVideo::image (bound to PlayerVideo::always_rgb) will take the source + * image and convert it (from whatever the user has said it is) to RGB. + */ + + _frame = pv->image ( + bind (&Log::dcp_log, _film->log().get(), _1, _2), + bind (&PlayerVideo::always_rgb, _1), + false, true + ); + + ImageChanged (pv); + + _position = time; + _inter_position = pv->inter_position (); + _inter_size = pv->inter_size (); + + refresh_panel (); } +void +FilmViewer::get () +{ + Image const * current = _frame.get (); + while (!_player->pass() && _frame.get() == current) {} +} + +void +FilmViewer::timer () +{ + DCPTime const frame = DCPTime::from_frames (1, _film->video_frame_rate ()); + + if ((_position + frame) >= _film->length ()) { + _play_button->SetValue (false); + check_play_state (); + } else { + get (); + } + + update_position_label (); + update_position_slider (); +} void FilmViewer::paint_panel () @@ -212,22 +293,30 @@ FilmViewer::paint_panel () dc.SetPen (p); dc.SetBrush (b); dc.DrawRectangle (0, _out_size.height, _panel_size.width, _panel_size.height - _out_size.height); - } -} + } + if (_outline_content->GetValue ()) { + wxPen p (wxColour (255, 0, 0), 2); + dc.SetPen (p); + dc.SetBrush (*wxTRANSPARENT_BRUSH); + dc.DrawRectangle (_inter_position.x, _inter_position.y, _inter_size.width, _inter_size.height); + } +} void FilmViewer::slider_moved () { - if (_film && _player) { - Time t = _slider->GetValue() * _film->length() / 4096; - /* Ensure that we hit the end of the film at the end of the slider */ - if (t >= _film->length ()) { - t = _film->length() - _film->video_frames_to_time (1); - } - _player->seek (t, false); - fetch_next_frame (); + if (!_film) { + return; } + + DCPTime t (_slider->GetValue() * _film->length().get() / 4096); + /* Ensure that we hit the end of the film at the end of the slider */ + if (t >= _film->length ()) { + t = _film->length() - DCPTime::from_frames (1, _film->video_frame_rate ()); + } + seek (t, false); + update_position_label (); } void @@ -235,8 +324,11 @@ FilmViewer::panel_sized (wxSizeEvent& ev) { _panel_size.width = ev.GetSize().GetWidth(); _panel_size.height = ev.GetSize().GetHeight(); + calculate_sizes (); - fetch_current_frame_again (); + refresh (); + update_position_label (); + update_position_slider (); } void @@ -247,18 +339,18 @@ FilmViewer::calculate_sizes () } Ratio const * container = _film->container (); - + float const panel_ratio = _panel_size.ratio (); float const film_ratio = container ? container->ratio () : 1.78; - + if (panel_ratio < film_ratio) { /* panel is less widscreen than the film; clamp width */ _out_size.width = _panel_size.width; - _out_size.height = _out_size.width / film_ratio; + _out_size.height = lrintf (_out_size.width / film_ratio); } else { /* panel is more widescreen than the film; clamp height */ _out_size.height = _panel_size.height; - _out_size.width = _out_size.height * film_ratio; + _out_size.width = lrintf (_out_size.height * film_ratio); } /* Catch silly values */ @@ -280,7 +372,7 @@ FilmViewer::check_play_state () if (!_film || _film->video_frame_rate() == 0) { return; } - + if (_play_button->GetValue()) { _timer.Start (1000 / _film->video_frame_rate()); } else { @@ -289,128 +381,196 @@ FilmViewer::check_play_state () } void -FilmViewer::process_video (shared_ptr pvf, Time t) +FilmViewer::update_position_slider () { - if (pvf->eyes() == EYES_RIGHT) { + if (!_film) { + _slider->SetValue (0); return; } - - _frame = pvf->image (); - _got_frame = true; - set_position_text (t); + DCPTime const len = _film->length (); + + if (len.get ()) { + int const new_slider_position = 4096 * _position.get() / len.get(); + if (new_slider_position != _slider->GetValue()) { + _slider->SetValue (new_slider_position); + } + } } void -FilmViewer::set_position_text (Time t) +FilmViewer::update_position_label () { if (!_film) { _frame_number->SetLabel ("0"); _timecode->SetLabel ("0:0:0.0"); return; } - + double const fps = _film->video_frame_rate (); /* Count frame number from 1 ... not sure if this is the best idea */ - _frame_number->SetLabel (wxString::Format (wxT("%d"), int (rint (t * fps / TIME_HZ)) + 1)); - - double w = static_cast(t) / TIME_HZ; - int const h = (w / 3600); - w -= h * 3600; - int const m = (w / 60); - w -= m * 60; - int const s = floor (w); - w -= s; - int const f = rint (w * fps); - _timecode->SetLabel (wxString::Format (wxT("%02d:%02d:%02d.%02d"), h, m, s, f)); + _frame_number->SetLabel (wxString::Format (wxT("%ld"), lrint (_position.seconds() * fps) + 1)); + _timecode->SetLabel (time_to_timecode (_position, fps)); } -/** Ask the player to emit its next frame, then update our display */ void -FilmViewer::fetch_next_frame () +FilmViewer::active_jobs_changed (optional j) { - /* Clear our frame in case we don't get a new one */ - _frame.reset (); + /* examine content is the only job which stops the viewer working */ + bool const a = !j || *j != "examine_content"; + _slider->Enable (a); + _play_button->Enable (a); +} - if (!_player) { - return; +DCPTime +FilmViewer::nudge_amount (wxMouseEvent& ev) +{ + DCPTime amount = DCPTime::from_frames (1, _film->video_frame_rate ()); + + if (ev.ShiftDown() && !ev.ControlDown()) { + amount = DCPTime::from_seconds (1); + } else if (!ev.ShiftDown() && ev.ControlDown()) { + amount = DCPTime::from_seconds (10); + } else if (ev.ShiftDown() && ev.ControlDown()) { + amount = DCPTime::from_seconds (60); } - _got_frame = false; - - try { - while (!_got_frame && !_player->pass ()) {} - } catch (DecodeError& e) { - _play_button->SetValue (false); - check_play_state (); - error_dialog (this, wxString::Format (_("Could not decode video for view (%s)"), std_to_wx(e.what()).data())); - } catch (OpenFileError& e) { - /* There was a problem opening a content file; we'll let this slide as it - probably means a missing content file, which we're already taking care of. - */ + return amount; +} + +void +FilmViewer::go_to (DCPTime t) +{ + if (t < DCPTime ()) { + t = DCPTime (); } - _panel->Refresh (); - _panel->Update (); + if (t >= _film->length ()) { + t = _film->length (); + } + + seek (t, true); + update_position_label (); + update_position_slider (); } void -FilmViewer::active_jobs_changed (bool a) +FilmViewer::back_clicked (wxMouseEvent& ev) { - if (a) { - list > jobs = JobManager::instance()->get (); - list >::iterator i = jobs.begin (); - while (i != jobs.end() && boost::dynamic_pointer_cast (*i) == 0) { - ++i; - } - - if (i == jobs.end() || (*i)->finished()) { - /* no examine content job running, so we're ok to use the viewer */ - a = false; - } - } - - _slider->Enable (!a); - _play_button->Enable (!a); + go_to (_position - nudge_amount (ev)); + ev.Skip (); } void -FilmViewer::back_clicked () +FilmViewer::forward_clicked (wxMouseEvent& ev) { - if (!_player) { + go_to (_position + nudge_amount (ev)); + ev.Skip (); +} + +void +FilmViewer::player_changed (bool frequent) +{ + if (frequent) { return; } - /* Player::video_position is the time after the last frame that we received. - We want to see the one before it, so we need to go back 2. - */ + if (_coalesce_player_changes) { + _pending_player_change = true; + return; + } + + calculate_sizes (); + refresh (); + update_position_label (); + update_position_slider (); +} + +void +FilmViewer::setup_sensitivity () +{ + bool const c = _film && !_film->content().empty (); + + _slider->Enable (c); + _back_button->Enable (c); + _forward_button->Enable (c); + _play_button->Enable (c); + _outline_content->Enable (c); + _frame_number->Enable (c); + _timecode->Enable (c); + + _left_eye->Enable (c && _film->three_d ()); + _right_eye->Enable (c && _film->three_d ()); +} - Time p = _player->video_position() - _film->video_frames_to_time (2); - if (p < 0) { - p = 0; +void +FilmViewer::film_changed (Film::Property p) +{ + if (p == Film::CONTENT || p == Film::THREE_D) { + setup_sensitivity (); } - - _player->seek (p, true); - fetch_next_frame (); } +/** Re-get the current frame */ void -FilmViewer::forward_clicked () +FilmViewer::refresh () { - if (!_player) { - return; + seek (_position, _last_seek_accurate); +} + +void +FilmViewer::set_position (DCPTime p) +{ + _position = p; + seek (p, true); + update_position_label (); + update_position_slider (); +} + +void +FilmViewer::set_coalesce_player_changes (bool c) +{ + _coalesce_player_changes = c; + + if (c) { + _pending_player_change = false; + } else { + if (_pending_player_change) { + player_changed (false); + } } +} - fetch_next_frame (); +void +FilmViewer::timecode_clicked () +{ + PlayheadToTimecodeDialog* dialog = new PlayheadToTimecodeDialog (this, _film->video_frame_rate ()); + if (dialog->ShowModal() == wxID_OK) { + go_to (dialog->get ()); + } + dialog->Destroy (); } void -FilmViewer::player_changed (bool frequent) +FilmViewer::frame_number_clicked () { - if (frequent) { - return; + PlayheadToFrameDialog* dialog = new PlayheadToFrameDialog (this, _film->video_frame_rate ()); + if (dialog->ShowModal() == wxID_OK) { + go_to (dialog->get ()); } + dialog->Destroy (); +} - calculate_sizes (); - fetch_current_frame_again (); +void +FilmViewer::jump_to_selected_clicked () +{ + Config::instance()->set_jump_to_selected (_jump_to_selected->GetValue ()); +} + +void +FilmViewer::seek (DCPTime t, bool accurate) +{ + _player->seek (t, accurate); + _last_seek_accurate = accurate; + get (); }