Tidy up the region menu a bit, and hopefully clarify what regions will be
[ardour.git] / gtk2_ardour / rhythm_ferret.cc
index 62f51c8a6599c847ef4527b744319791c5fe2337..da31feca3c8fc6f81f3ae3f7489138fbec86301d 100644 (file)
@@ -2,6 +2,7 @@
 #include <gtkmm2ext/utils.h>
 
 #include "pbd/memento_command.h"
+#include "pbd/convert.h"
 
 #include "ardour/transient_detector.h"
 #include "ardour/onset_detector.h"
@@ -13,8 +14,9 @@
 
 #include "rhythm_ferret.h"
 #include "audio_region_view.h"
-#include "public_editor.h"
+#include "editor.h"
 #include "utils.h"
+#include "time_axis_view.h"
 
 #include "i18n.h"
 
@@ -44,162 +46,110 @@ static const gchar * _onset_function_strings[] = {
        0
 };
 
-RhythmFerret::RhythmFerret (PublicEditor& e)
+static const gchar * _operation_strings[] = {
+       N_("Split region"),
+       N_("Set tempo map"),
+       N_("Conform region"),
+       0
+};
+
+RhythmFerret::RhythmFerret (Editor& e)
        : ArdourDialog (_("Rhythm Ferret"))
        , editor (e)
-       , operation_frame (_("Operation"))
-       , selection_frame (_("Selection"))
-       , ferret_frame (_("Analysis"))
-       , logo (0)
-       , region_split_button (operation_button_group, _("Split region"))
-       , tempo_button (operation_button_group, _("Set tempo map"))
-       , region_conform_button (operation_button_group, _("Conform region"))
-       , analysis_mode_label (_("Mode"))
        , detection_threshold_adjustment (3, 0, 20, 1, 4)
        , detection_threshold_scale (detection_threshold_adjustment)
-       , detection_threshold_label (_("Threshold"))
        , sensitivity_adjustment (40, 0, 100, 1, 10)
        , sensitivity_scale (sensitivity_adjustment)
-       , sensitivity_label (_("Sensitivity"))
        , analyze_button (_("Analyze"))
-       , onset_function_label (_("Detection function"))
        , peak_picker_threshold_adjustment (0.3, 0.0, 1.0, 0.01, 0.1)
        , peak_picker_threshold_scale (peak_picker_threshold_adjustment)
-       , peak_picker_label (_("Peak Threshold"))
        , silence_threshold_adjustment (-90.0, -120.0, 0.0, 1, 10)
        , silence_threshold_scale (silence_threshold_adjustment)
-       , silence_label (_("Silent Threshold (dB)"))
        , trigger_gap_adjustment (3, 0, 100, 1, 10)
        , trigger_gap_spinner (trigger_gap_adjustment)
-       , trigger_gap_label (_("Trigger gap (msecs)"))
        , action_button (Stock::APPLY)
-
 {
-       upper_hpacker.set_spacing (6);
-
-       upper_hpacker.pack_start (ferret_frame, true, true);
-       upper_hpacker.pack_start (selection_frame, true, true);
-       upper_hpacker.pack_start (operation_frame, true, true);
-
-       op_packer.pack_start (region_split_button, false, false);
-       op_packer.pack_start (tempo_button, false, false);
-       op_packer.pack_start (region_conform_button, false, false);
-
-       operation_frame.add (op_packer);
-
-       HBox* box;
-
-       ferret_packer.set_spacing (6);
-       ferret_packer.set_border_width (6);
-       
-       vector<string> strings;
+       operation_strings = I18N (_operation_strings);
+       Gtkmm2ext::set_popdown_strings (operation_selector, operation_strings);
+       operation_selector.set_active (0);
 
        analysis_mode_strings = I18N (_analysis_mode_strings);
        Gtkmm2ext::set_popdown_strings (analysis_mode_selector, analysis_mode_strings);
        analysis_mode_selector.set_active_text (analysis_mode_strings.front());
-       analysis_mode_selector.signal_changed().connect (mem_fun (*this, &RhythmFerret::analysis_mode_changed));
+       analysis_mode_selector.signal_changed().connect (sigc::mem_fun (*this, &RhythmFerret::analysis_mode_changed));
 
        onset_function_strings = I18N (_onset_function_strings);
        Gtkmm2ext::set_popdown_strings (onset_detection_function_selector, onset_function_strings);
-       /* Onset plugin uses complex domain as default function 
+       /* Onset plugin uses complex domain as default function
           XXX there should be a non-hacky way to set this
         */
        onset_detection_function_selector.set_active_text (onset_function_strings[3]);
 
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (analysis_mode_label, false, false);
-       box->pack_start (analysis_mode_selector, true, true);
-       ferret_packer.pack_start (*box, false, false);
+       Table* t = manage (new Table (7, 3));
+       t->set_spacings (12);
 
-       ferret_packer.pack_start (analysis_packer, false, false);
+       int n = 0;
 
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (trigger_gap_label, false, false);
-       box->pack_start (trigger_gap_spinner, false, false);
-       ferret_packer.pack_start (*box, false, false);
+       t->attach (*manage (new Label (_("Mode"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (analysis_mode_selector, 1, 2, n, n + 1, FILL);
+       ++n;
 
-       ferret_packer.pack_start (analyze_button, false, false);
+       t->attach (*manage (new Label (_("Detection function"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (onset_detection_function_selector, 1, 2, n, n + 1, FILL);
+       ++n;
 
-       analyze_button.signal_clicked().connect (mem_fun (*this, &RhythmFerret::run_analysis));
-       
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (detection_threshold_label, false, false);
-       box->pack_start (detection_threshold_scale, true, true);
-       perc_onset_packer.pack_start (*box, false, false);
-               
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (sensitivity_label, false, false);
-       box->pack_start (sensitivity_scale, true, true);
-       perc_onset_packer.pack_start (*box, false, false);
-
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (onset_function_label, false, false);
-       box->pack_start (onset_detection_function_selector, true, true);
-       note_onset_packer.pack_start (*box, false, false);
-               
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (peak_picker_label, false, false);
-       box->pack_start (peak_picker_threshold_scale, true, true);
-       note_onset_packer.pack_start (*box, false, false);
-       
-       box = manage (new HBox);
-       box->set_spacing (6);
-       box->pack_start (silence_label, false, false);
-       box->pack_start (silence_threshold_scale, true, true);
-       note_onset_packer.pack_start (*box, false, false);
+       t->attach (*manage (new Label (_("Trigger gap"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (trigger_gap_spinner, 1, 2, n, n + 1, FILL);
+       t->attach (*manage (new Label (_("ms"))), 2, 3, n, n + 1, FILL);
+       ++n;
 
-       analysis_mode_changed ();
+       t->attach (*manage (new Label (_("Threshold"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (detection_threshold_scale, 1, 2, n, n + 1, FILL);
+       t->attach (*manage (new Label (_("dB"))), 2, 3, n, n + 1, FILL);
+       ++n;
 
-       ferret_frame.add (ferret_packer);
+       t->attach (*manage (new Label (_("Peak threshold"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (peak_picker_threshold_scale, 1, 2, n, n + 1, FILL);
+       t->attach (*manage (new Label (_("dB"))), 2, 3, n, n + 1, FILL);
+       ++n;
        
-       logo = manage (new Gtk::Image (::get_icon (X_("ferret_02"))));
+       t->attach (*manage (new Label (_("Silence threshold"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (silence_threshold_scale, 1, 2, n, n + 1, FILL);
+       t->attach (*manage (new Label (_("dB"))), 2, 3, n, n + 1, FILL);
+       ++n;
 
-       if (logo) {
-               lower_hpacker.pack_start (*logo, false, false);
-       }
+       t->attach (*manage (new Label (_("Sensitivity"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (sensitivity_scale, 1, 2, n, n + 1, FILL);
+       ++n;
 
-       lower_hpacker.pack_start (operation_clarification_label, true, true);
-       lower_hpacker.pack_start (action_button, false, false);
-       lower_hpacker.set_border_width (6);
-       lower_hpacker.set_spacing (6);
+       t->attach (*manage (new Label (_("Operation"), 1, 0.5)), 0, 1, n, n + 1, FILL);
+       t->attach (operation_selector, 1, 2, n, n + 1, FILL);
+       ++n;
+
+       analyze_button.signal_clicked().connect (sigc::mem_fun (*this, &RhythmFerret::run_analysis));
+       action_button.signal_clicked().connect (sigc::mem_fun (*this, &RhythmFerret::do_action));
 
-       action_button.signal_clicked().connect (mem_fun (*this, &RhythmFerret::do_action));
-       
        get_vbox()->set_border_width (6);
        get_vbox()->set_spacing (6);
-       get_vbox()->pack_start (upper_hpacker, true, true);
-       get_vbox()->pack_start (lower_hpacker, false, false);
+       get_vbox()->pack_start (*t);
 
-       show_all ();
-}
+       add_action_widget (analyze_button, 1);
+       add_action_widget (action_button, 0);
 
-RhythmFerret::~RhythmFerret()
-{
-       delete logo;
+       show_all ();
+       analysis_mode_changed ();
 }
 
 void
 RhythmFerret::analysis_mode_changed ()
 {
-       analysis_packer.children().clear ();
-
-       switch (get_analysis_mode()) {
-       case PercussionOnset:
-               analysis_packer.pack_start (perc_onset_packer);
-               break;
-
-       case NoteOnset:
-               analysis_packer.pack_start (note_onset_packer);
-               break;
-       }
-
-       analysis_packer.show_all ();
+       bool const perc = get_analysis_mode() == PercussionOnset;
+       
+       detection_threshold_scale.set_sensitive (perc);
+       sensitivity_scale.set_sensitive (perc);
+       onset_detection_function_selector.set_sensitive (!perc);
+       peak_picker_threshold_scale.set_sensitive (!perc);
+       silence_threshold_scale.set_sensitive (!perc);
 }
 
 RhythmFerret::AnalysisMode
@@ -209,7 +159,7 @@ RhythmFerret::get_analysis_mode () const
 
        if (str == analysis_mode_strings[(int) NoteOnset]) {
                return NoteOnset;
-       } 
+       }
 
        return PercussionOnset;
 }
@@ -217,9 +167,9 @@ RhythmFerret::get_analysis_mode () const
 RhythmFerret::Action
 RhythmFerret::get_action () const
 {
-       if (tempo_button.get_active()) {
+       if (operation_selector.get_active_row_number() == 1) {
                return DefineTempoMap;
-       } else if (region_conform_button.get_active()) {
+       } else if (operation_selector.get_active_row_number() == 2) {
                return ConformRegion;
        }
 
@@ -229,7 +179,7 @@ RhythmFerret::get_action () const
 void
 RhythmFerret::run_analysis ()
 {
-       if (!session) {
+       if (!_session) {
                return;
        }
 
@@ -255,19 +205,16 @@ RhythmFerret::run_analysis ()
                default:
                        break;
                }
-
-       }
-
-       for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ++i) {
-               (*i)->get_time_axis_view().show_feature_lines (current_results);
+               
+               (*i)->region()->set_transients (current_results);
+               current_results.clear();
        }
-
 }
 
 int
-RhythmFerret::run_percussion_onset_analysis (boost::shared_ptr<Readable> readable, nframes64_t offset, AnalysisFeatureList& results)
+RhythmFerret::run_percussion_onset_analysis (boost::shared_ptr<Readable> readable, framepos_t /*offset*/, AnalysisFeatureList& results)
 {
-       TransientDetector t (session->frame_rate());
+       TransientDetector t (_session->frame_rate());
 
        for (uint32_t i = 0; i < readable->n_channels(); ++i) {
 
@@ -283,18 +230,18 @@ RhythmFerret::run_percussion_onset_analysis (boost::shared_ptr<Readable> readabl
 
                /* translate all transients to give absolute position */
 
-               for (AnalysisFeatureList::iterator x = these_results.begin(); x != these_results.end(); ++x) {
-                       (*x) += offset;
-               }
+               //for (AnalysisFeatureList::iterator x = these_results.begin(); x != these_results.end(); ++x) {
+               //      (*x) += offset;
+               //}
 
                /* merge */
-               
+
                results.insert (results.end(), these_results.begin(), these_results.end());
                these_results.clear ();
        }
 
        if (!results.empty()) {
-               TransientDetector::cleanup_transients (results, session->frame_rate(), trigger_gap_adjustment.get_value());
+               TransientDetector::cleanup_transients (results, _session->frame_rate(), trigger_gap_adjustment.get_value());
        }
 
        return 0;
@@ -318,33 +265,33 @@ RhythmFerret::get_note_onset_function ()
 }
 
 int
-RhythmFerret::run_note_onset_analysis (boost::shared_ptr<Readable> readable, nframes64_t offset, AnalysisFeatureList& results)
+RhythmFerret::run_note_onset_analysis (boost::shared_ptr<Readable> readable, framepos_t /*offset*/, AnalysisFeatureList& results)
 {
        try {
-               OnsetDetector t (session->frame_rate());
-               
+               OnsetDetector t (_session->frame_rate());
+
                for (uint32_t i = 0; i < readable->n_channels(); ++i) {
-                       
+
                        AnalysisFeatureList these_results;
-                       
+
                        t.reset ();
-                       
+
                        t.set_function (get_note_onset_function());
                        t.set_silence_threshold (silence_threshold_adjustment.get_value());
                        t.set_peak_threshold (peak_picker_threshold_adjustment.get_value());
-                       
+
                        if (t.run ("", readable.get(), i, these_results)) {
                                continue;
                        }
-                       
+
                        /* translate all transients to give absolute position */
-                       
-                       for (AnalysisFeatureList::iterator x = these_results.begin(); x != these_results.end(); ++x) {
-                               (*x) += offset;
-                       }
-                       
+
+                       //for (AnalysisFeatureList::iterator x = these_results.begin(); x != these_results.end(); ++x) {
+                       //      (*x) += offset;
+                       //}
+
                        /* merge */
-                       
+
                        results.insert (results.end(), these_results.begin(), these_results.end());
                        these_results.clear ();
                }
@@ -355,7 +302,7 @@ RhythmFerret::run_note_onset_analysis (boost::shared_ptr<Readable> readable, nfr
        }
 
        if (!results.empty()) {
-               OnsetDetector::cleanup_onsets (results, session->frame_rate(), trigger_gap_adjustment.get_value());
+               OnsetDetector::cleanup_onsets (results, _session->frame_rate(), trigger_gap_adjustment.get_value());
        }
 
        return 0;
@@ -364,7 +311,7 @@ RhythmFerret::run_note_onset_analysis (boost::shared_ptr<Readable> readable, nfr
 void
 RhythmFerret::do_action ()
 {
-       if (!session || current_results.empty()) {
+       if (!_session) {
                return;
        }
 
@@ -372,7 +319,9 @@ RhythmFerret::do_action ()
        case SplitRegion:
                do_split_action ();
                break;
-
+       case ConformRegion:
+               editor.close_region_gaps();
+               break;
        default:
                break;
        }
@@ -381,15 +330,27 @@ RhythmFerret::do_action ()
 void
 RhythmFerret::do_split_action ()
 {
-       /* this can/will change the current selection, so work with a copy */
-
        RegionSelection& regions (editor.get_selection().regions);
-
+       
        if (regions.empty()) {
                return;
        }
 
-       session->begin_reversible_command (_("split regions (rhythm ferret)"));
+       _session->begin_reversible_command (_("split regions (rhythm ferret)"));
+       
+       /* Merge the transient positions for regions in consideration */
+       AnalysisFeatureList merged_features;
+       
+       for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ++i) {
+               
+               AnalysisFeatureList features;
+               features = (*i)->region()->transients();
+               
+               merged_features.insert (merged_features.end(), features.begin(), features.end());               
+       }
+       
+       merged_features.sort();
+       merged_features.unique();
 
        for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ) {
 
@@ -398,16 +359,16 @@ RhythmFerret::do_split_action ()
                tmp = i;
                ++tmp;
 
-               (*i)->get_time_axis_view().hide_feature_lines ();
-
-               editor.split_region_at_points ((*i)->region(), current_results, false);
+               AnalysisFeatureList features;
+               features = (*i)->region()->transients();
+               editor.split_region_at_points ((*i)->region(), merged_features, false);
 
                /* i is invalid at this point */
 
                i = tmp;
        }
-       
-       session->commit_reversible_command ();
+
+       _session->commit_reversible_command ();
 }
 
 void
@@ -417,15 +378,9 @@ RhythmFerret::set_session (Session* s)
        current_results.clear ();
 }
 
-static void hide_time_axis_features (TimeAxisView& tav)
-{
-       tav.hide_feature_lines ();
-}
-
 void
 RhythmFerret::on_hide ()
 {
-       editor.foreach_time_axis_view (sigc::ptr_fun (hide_time_axis_features));
        ArdourDialog::on_hide ();
 }