X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Fwx%2Faudio_dialog.cc;h=2f1f1c8265f324f41cb1c2733dd0f02ad44267bc;hp=c753de535cb8d49f1e5e74d7ba7cc0abd4a868a1;hb=39dcdd18487d5d1e20f0343fe617ed5bf44c1387;hpb=1dd9fb73e28f079fb0d4ee46ab3e18008e42aca2 diff --git a/src/wx/audio_dialog.cc b/src/wx/audio_dialog.cc index c753de535..2f1f1c826 100644 --- a/src/wx/audio_dialog.cc +++ b/src/wx/audio_dialog.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2013-2016 Carl Hetherington + Copyright (C) 2013-2018 Carl Hetherington This file is part of DCP-o-matic. @@ -21,6 +21,8 @@ #include "audio_dialog.h" #include "audio_plot.h" #include "wx_util.h" +#include "static_text.h" +#include "check_box.h" #include "lib/audio_analysis.h" #include "lib/film.h" #include "lib/analyse_audio_job.h" @@ -33,13 +35,19 @@ using std::cout; using std::list; +using std::vector; +using std::pair; using boost::shared_ptr; using boost::bind; using boost::optional; using boost::const_pointer_cast; using boost::dynamic_pointer_cast; +using namespace dcpomatic; -/** @param content Content to analyse, or 0 to analyse all of the film's audio */ +/** @param parent Parent window. + * @param film Film we are using. + * @param content Content to analyse, or 0 to analyse all of the film's audio. + */ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr film, shared_ptr content) : wxDialog ( parent, @@ -47,7 +55,14 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr film, shared_ptr film, shared_ptrAdd (_cursor, 0, wxTOP, DCPOMATIC_SIZER_Y_GAP); _plot = new AudioPlot (this); left->Add (_plot, 1, wxTOP | wxEXPAND, 12); - _sample_peak = new wxStaticText (this, wxID_ANY, wxT ("")); + _sample_peak = new StaticText (this, wxT ("")); left->Add (_sample_peak, 0, wxTOP, DCPOMATIC_SIZER_Y_GAP); - _true_peak = new wxStaticText (this, wxID_ANY, wxT ("")); + _true_peak = new StaticText (this, wxT ("")); left->Add (_true_peak, 0, wxTOP, DCPOMATIC_SIZER_Y_GAP); - _integrated_loudness = new wxStaticText (this, wxID_ANY, wxT ("")); + _integrated_loudness = new StaticText (this, wxT ("")); left->Add (_integrated_loudness, 0, wxTOP, DCPOMATIC_SIZER_Y_GAP); - _loudness_range = new wxStaticText (this, wxID_ANY, wxT ("")); + _loudness_range = new StaticText (this, wxT ("")); left->Add (_loudness_range, 0, wxTOP, DCPOMATIC_SIZER_Y_GAP); - lr_sizer->Add (left, 1, wxALL, 12); + lr_sizer->Add (left, 1, wxALL | wxEXPAND, 12); wxBoxSizer* right = new wxBoxSizer (wxVERTICAL); { - wxStaticText* m = new wxStaticText (this, wxID_ANY, _("Channels")); + wxStaticText* m = new StaticText (this, _("Channels")); m->SetFont (subheading_font); right->Add (m, 1, wxALIGN_CENTER_VERTICAL | wxTOP | wxBOTTOM, 16); } - for (int i = 0; i < _channels; ++i) { - _channel_checkbox[i] = new wxCheckBox (this, wxID_ANY, std_to_wx (audio_channel_name (i))); + for (int i = 0; i < MAX_DCP_AUDIO_CHANNELS; ++i) { + _channel_checkbox[i] = new CheckBox (this, std_to_wx(audio_channel_name(i))); + _channel_checkbox[i]->SetForegroundColour(wxColour(_plot->colour(i))); right->Add (_channel_checkbox[i], 0, wxEXPAND | wxALL, 3); - _channel_checkbox[i]->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&AudioDialog::channel_clicked, this, _1)); + _channel_checkbox[i]->Bind (wxEVT_CHECKBOX, boost::bind (&AudioDialog::channel_clicked, this, _1)); } { - wxStaticText* m = new wxStaticText (this, wxID_ANY, _("Type")); + wxStaticText* m = new StaticText (this, _("Type")); m->SetFont (subheading_font); right->Add (m, 1, wxALIGN_CENTER_VERTICAL | wxTOP, 16); } @@ -101,13 +119,13 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr film, shared_ptrAdd (_type_checkbox[i], 0, wxEXPAND | wxALL, 3); - _type_checkbox[i]->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&AudioDialog::type_clicked, this, _1)); + _type_checkbox[i]->Bind (wxEVT_CHECKBOX, boost::bind (&AudioDialog::type_clicked, this, _1)); } { - wxStaticText* m = new wxStaticText (this, wxID_ANY, _("Smoothing")); + wxStaticText* m = new StaticText (this, _("Smoothing")); m->SetFont (subheading_font); right->Add (m, 1, wxALIGN_CENTER_VERTICAL | wxTOP, 16); } @@ -118,7 +136,7 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr film, shared_ptrAdd (right, 0, wxALL, 12); - overall_sizer->Add (lr_sizer); + overall_sizer->Add (lr_sizer, 0, wxEXPAND); #ifdef DCPOMATIC_LINUX wxSizer* buttons = CreateSeparatedButtonSizer (wxCLOSE); @@ -131,15 +149,19 @@ AudioDialog::AudioDialog (wxWindow* parent, shared_ptr film, shared_ptrLayout (); overall_sizer->SetSizeHints (this); - _film_connection = film->ContentChanged.connect (boost::bind (&AudioDialog::content_changed, this, _2)); - SetTitle (_("DCP-o-matic audio")); + _film_connection = film->Change.connect (boost::bind(&AudioDialog::film_change, this, _1, _2)); + _film_content_connection = film->ContentChange.connect (boost::bind (&AudioDialog::content_change, this, _1, _3)); + DCPOMATIC_ASSERT (film->directory()); + SetTitle(wxString::Format(_("DCP-o-matic audio - %s"), std_to_wx(film->directory().get().string()))); if (content) { _playlist.reset (new Playlist ()); - const_pointer_cast (_playlist)->add (content); + const_pointer_cast(_playlist)->add(film, content); } else { _playlist = film->playlist (); } + + _plot->Cursor.connect (bind (&AudioDialog::set_cursor, this, _1, _2)); } void @@ -152,19 +174,38 @@ AudioDialog::try_to_load_analysis () shared_ptr film = _film.lock (); DCPOMATIC_ASSERT (film); + shared_ptr check = _content.lock(); + boost::filesystem::path const path = film->audio_analysis_path (_playlist); if (!boost::filesystem::exists (path)) { _plot->set_analysis (shared_ptr ()); _analysis.reset (); - JobManager::instance()->analyse_audio (film, _playlist, _analysis_finished_connection, bind (&AudioDialog::analysis_finished, this)); + + BOOST_FOREACH (shared_ptr i, JobManager::instance()->get()) { + if (dynamic_pointer_cast(i)) { + i->cancel (); + } + } + + JobManager::instance()->analyse_audio ( + film, _playlist, !static_cast(check), _analysis_finished_connection, bind (&AudioDialog::analysis_finished, this) + ); return; } try { _analysis.reset (new AudioAnalysis (path)); + } catch (OldFormatError& e) { + /* An old analysis file: recreate it */ + JobManager::instance()->analyse_audio ( + film, _playlist, !static_cast(check), _analysis_finished_connection, bind (&AudioDialog::analysis_finished, this) + ); + return; } catch (xmlpp::exception& e) { - /* Probably an old-style analysis file: recreate it */ - JobManager::instance()->analyse_audio (film, _playlist, _analysis_finished_connection, bind (&AudioDialog::analysis_finished, this)); + /* Probably a (very) old-style analysis file: recreate it */ + JobManager::instance()->analyse_audio ( + film, _playlist, !static_cast(check), _analysis_finished_connection, bind (&AudioDialog::analysis_finished, this) + ); return; } @@ -172,6 +213,14 @@ AudioDialog::try_to_load_analysis () _plot->set_gain_correction (_analysis->gain_correction (_playlist)); setup_statistics (); + for (int i = 0; i < _channels; ++i) { + _channel_checkbox[i]->Show (); + } + + for (int i = _channels; i < MAX_DCP_AUDIO_CHANNELS; ++i) { + _channel_checkbox[i]->Hide (); + } + /* Set up some defaults if no check boxes are checked */ int i = 0; @@ -248,8 +297,28 @@ AudioDialog::channel_clicked (wxCommandEvent& ev) } void -AudioDialog::content_changed (int p) +AudioDialog::film_change (ChangeType type, int p) { + if (type != CHANGE_TYPE_DONE) { + return; + } + + if (p == Film::AUDIO_CHANNELS) { + shared_ptr film = _film.lock (); + if (film) { + _channels = film->audio_channels (); + try_to_load_analysis (); + } + } +} + +void +AudioDialog::content_change (ChangeType type, int p) +{ + if (type != CHANGE_TYPE_DONE) { + return; + } + if (p == AudioContentProperty::STREAMS) { try_to_load_analysis (); } else if (p == AudioContentProperty::GAIN) { @@ -296,27 +365,26 @@ AudioDialog::setup_statistics () return; } - if (static_cast(_analysis->sample_peak ())) { - - float const peak_dB = 20 * log10 (_analysis->sample_peak().get()) + _analysis->gain_correction (_playlist); - - _sample_peak->SetLabel ( - wxString::Format ( - _("Sample peak is %.2fdB at %s"), - peak_dB, - time_to_timecode (_analysis->sample_peak_time().get(), film->video_frame_rate ()).data () - ) - ); - - if (peak_dB > -3) { - _sample_peak->SetForegroundColour (wxColour (255, 0, 0)); - } else { - _sample_peak->SetForegroundColour (wxColour (0, 0, 0)); - } + pair const peak = _analysis->overall_sample_peak (); + float const peak_dB = linear_to_db(peak.first.peak) + _analysis->gain_correction(_playlist); + _sample_peak->SetLabel ( + wxString::Format ( + _("Sample peak is %.2fdB at %s on %s"), + peak_dB, + time_to_timecode (peak.first.time, film->video_frame_rate ()).data (), + std_to_wx (short_audio_channel_name (peak.second)).data () + ) + ); + + if (peak_dB > -3) { + _sample_peak->SetForegroundColour (wxColour (255, 0, 0)); + } else { + _sample_peak->SetForegroundColour (wxColour (0, 0, 0)); } - if (static_cast(_analysis->true_peak ())) { - float const peak_dB = 20 * log10 (_analysis->true_peak().get()) + _analysis->gain_correction (_playlist); + if (_analysis->overall_true_peak()) { + float const peak = _analysis->overall_true_peak().get(); + float const peak_dB = linear_to_db(peak) + _analysis->gain_correction(_playlist); _true_peak->SetLabel (wxString::Format (_("True peak is %.2fdB"), peak_dB)); @@ -355,3 +423,16 @@ AudioDialog::Show (bool show) try_to_load_analysis (); return r; } + +void +AudioDialog::set_cursor (optional time, optional db) +{ + if (!time || !db) { + _cursor->SetLabel (_("Cursor: none")); + return; + } + + shared_ptr film = _film.lock(); + DCPOMATIC_ASSERT (film); + _cursor->SetLabel (wxString::Format (_("Cursor: %.1fdB at %s"), *db, time->timecode(film->video_frame_rate()))); +}