Use rectified log waveform in strip silence dialogue. Add threshold graphical indication. Update dialogue on open (fixes #3102). Various other small cleanups.

git-svn-id: svn://localhost/ardour2/branches/3.0@7030 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Carl Hetherington 2010-04-30 14:50:25 +00:00
parent 7df6278865
commit ea21e628e0
2 changed files with 176 additions and 102 deletions

View file

@ -32,10 +32,12 @@
#include "gui_thread.h" #include "gui_thread.h"
#include "strip_silence_dialog.h" #include "strip_silence_dialog.h"
#include "canvas_impl.h" #include "canvas_impl.h"
#include "simpleline.h"
#include "waveview.h" #include "waveview.h"
#include "simplerect.h" #include "simplerect.h"
#include "rgb_macros.h" #include "rgb_macros.h"
#include "i18n.h" #include "i18n.h"
#include "logmeter.h"
using namespace ARDOUR; using namespace ARDOUR;
using namespace std; using namespace std;
@ -49,7 +51,7 @@ InterThreadInfo StripSilenceDialog::itt;
StripSilenceDialog* StripSilenceDialog::current = 0; StripSilenceDialog* StripSilenceDialog::current = 0;
/** Construct Strip silence dialog box */ /** Construct Strip silence dialog box */
StripSilenceDialog::StripSilenceDialog (Session* s, std::list<boost::shared_ptr<ARDOUR::AudioRegion> > const & regions) StripSilenceDialog::StripSilenceDialog (Session* s, list<boost::shared_ptr<ARDOUR::AudioRegion> > const & regions)
: ArdourDialog (_("Strip Silence")) : ArdourDialog (_("Strip Silence"))
, _minimum_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false) , _minimum_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false)
, _fade_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false) , _fade_length (X_("silence duration"), true, "SilenceDurationClock", true, false, true, false)
@ -65,92 +67,104 @@ StripSilenceDialog::StripSilenceDialog (Session* s, std::list<boost::shared_ptr<
thread_run = new Glib::Cond; thread_run = new Glib::Cond;
} }
for (std::list<boost::shared_ptr<ARDOUR::AudioRegion> >::const_iterator i = regions.begin(); i != regions.end(); ++i) {
Wave w;
w.region = *i;
_waves.push_back (w);
}
Gtk::HBox* hbox = Gtk::manage (new Gtk::HBox); Gtk::HBox* hbox = Gtk::manage (new Gtk::HBox);
hbox->set_spacing (16);
Gtk::Table* table = Gtk::manage (new Gtk::Table (4, 3)); Gtk::Table* table = Gtk::manage (new Gtk::Table (3, 3));
table->set_spacings (4); table->set_spacings (6);
Gtk::Label* l = Gtk::manage (new Gtk::Label (_("Threshold:"))); int n = 0;
l->set_alignment (1, 0.5);
hbox->pack_start (*l, false, false); table->attach (*Gtk::manage (new Gtk::Label (_("Threshold"), 1, 0.5)), 0, 1, n, n + 1, Gtk::FILL);
hbox->pack_start (_threshold, true, true); table->attach (_threshold, 1, 2, n, n + 1, Gtk::FILL);
table->attach (*Gtk::manage (new Gtk::Label (_("dbFS"))), 2, 3, n, n + 1, Gtk::FILL);
++n;
_threshold.set_digits (1); _threshold.set_digits (1);
_threshold.set_increments (1, 10); _threshold.set_increments (1, 10);
_threshold.set_range (-120, 0); _threshold.set_range (-120, 0);
_threshold.set_value (-60); _threshold.set_value (-60);
l = Gtk::manage (new Gtk::Label (_("dBFS"))); table->attach (*Gtk::manage (new Gtk::Label (_("Minimum length"), 1, 0.5)), 0, 1, n, n + 1, Gtk::FILL);
l->set_alignment (0, 0.5); table->attach (_minimum_length, 1, 2, n, n + 1, Gtk::FILL);
++n;
hbox->pack_start (*l, false, false);
l = Gtk::manage (new Gtk::Label (_("Minimum length:")));
l->set_alignment (1, 0.5);
hbox->pack_start (*l, false, false);
hbox->pack_start (_minimum_length, true, true);
_minimum_length.set_session (s); _minimum_length.set_session (s);
_minimum_length.set_mode (AudioClock::Frames); _minimum_length.set_mode (AudioClock::Frames);
_minimum_length.set (1000, true); _minimum_length.set (1000, true);
l = Gtk::manage (new Gtk::Label (_("Fade length:"))); table->attach (*Gtk::manage (new Gtk::Label (_("Fade length"), 1, 0.5)), 0, 1, n, n + 1, Gtk::FILL);
l->set_alignment (1, 0.5); table->attach (_fade_length, 1, 2, n, n + 1, Gtk::FILL);
++n;
hbox->pack_start (*l, false, false);
hbox->pack_start (_fade_length, true, true);
_fade_length.set_session (s); _fade_length.set_session (s);
_fade_length.set_mode (AudioClock::Frames); _fade_length.set_mode (AudioClock::Frames);
_fade_length.set (64, true); _fade_length.set (64, true);
_segment_count_label.set_text (_("Silent segments: none")); hbox->pack_start (*table);
table = Gtk::manage (new Gtk::Table (3, 2));
table->set_spacings (6);
n = 0;
table->attach (*Gtk::manage (new Gtk::Label (_("Silent segments:"), 1, 0.5)), 3, 4, n, n + 1, Gtk::FILL);
table->attach (_segment_count_label, 5, 6, n, n + 1, Gtk::FILL);
_segment_count_label.set_alignment (0, 0.5);
++n;
table->attach (*Gtk::manage (new Gtk::Label (_("Shortest silence:"), 1, 0.5)), 3, 4, n, n + 1, Gtk::FILL);
table->attach (_shortest_silence_label, 5, 6, n, n + 1, Gtk::FILL);
_shortest_silence_label.set_alignment (0, 0.5);
++n;
table->attach (*Gtk::manage (new Gtk::Label (_("Shortest audible:"), 1, 0.5)), 3, 4, n, n + 1, Gtk::FILL);
table->attach (_shortest_audible_label, 5, 6, n, n + 1, Gtk::FILL);
_shortest_audible_label.set_alignment (0, 0.5);
++n;
hbox->pack_start (*table);
/* dummy label for padding */
hbox->pack_start (*Gtk::manage (new Gtk::Label ("")), true, true);
get_vbox()->pack_start (*hbox, false, false); get_vbox()->pack_start (*hbox, false, false);
add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL); add_button (Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
add_button (Gtk::Stock::APPLY, Gtk::RESPONSE_OK); add_button (Gtk::Stock::APPLY, Gtk::RESPONSE_OK);
_canvas = new CanvasAA (); _canvas = new CanvasAA ();
_canvas->signal_size_allocate().connect (sigc::mem_fun (*this, &StripSilenceDialog::canvas_allocation)); _canvas->signal_size_allocate().connect (sigc::mem_fun (*this, &StripSilenceDialog::canvas_allocation));
_canvas->set_size_request (_wave_width, _wave_height * _waves.size ()); _canvas->set_size_request (_wave_width, _wave_height * regions.size());
for (list<boost::shared_ptr<ARDOUR::AudioRegion> >::const_iterator i = regions.begin(); i != regions.end(); ++i) {
Wave* w = new Wave (_canvas->root(), *i);
_waves.push_back (w);
}
get_vbox()->pack_start (*_canvas, true, true); get_vbox()->pack_start (*_canvas, true, true);
get_vbox()->pack_start (_segment_count_label, false, false);
show_all (); show_all ();
_threshold.get_adjustment()->signal_value_changed().connect (sigc::mem_fun (*this, &StripSilenceDialog::maybe_start_silence_detection)); _threshold.get_adjustment()->signal_value_changed().connect (sigc::mem_fun (*this, &StripSilenceDialog::threshold_changed));
_minimum_length.ValueChanged.connect (sigc::mem_fun (*this, &StripSilenceDialog::maybe_start_silence_detection)); _minimum_length.ValueChanged.connect (sigc::mem_fun (*this, &StripSilenceDialog::maybe_start_silence_detection));
create_waves (); create_waves ();
update_silence_rects (); update_silence_rects ();
update_threshold_line ();
maybe_start_silence_detection ();
} }
StripSilenceDialog::~StripSilenceDialog () StripSilenceDialog::~StripSilenceDialog ()
{ {
for (std::list<Wave>::iterator i = _waves.begin(); i != _waves.end(); ++i) { for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
delete i->view; delete *i;
for (std::list<SimpleRect*>::iterator j = i->silence_rects.begin(); j != i->silence_rects.end(); ++j) {
delete *j;
}
} }
delete _peaks_ready_connection; _waves.clear ();
delete _peaks_ready_connection;
delete _canvas; delete _canvas;
} }
@ -162,23 +176,25 @@ StripSilenceDialog::create_waves ()
delete _peaks_ready_connection; delete _peaks_ready_connection;
_peaks_ready_connection = 0; _peaks_ready_connection = 0;
for (std::list<Wave>::iterator i = _waves.begin(); i != _waves.end(); ++i) { for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
if (i->region->audio_source(0)->peaks_ready (boost::bind (&StripSilenceDialog::peaks_ready, this), &_peaks_ready_connection, gui_context())) { if ((*i)->region->audio_source(0)->peaks_ready (boost::bind (&StripSilenceDialog::peaks_ready, this), &_peaks_ready_connection, gui_context())) {
i->view = new WaveView (*(_canvas->root())); (*i)->view = new WaveView (*(_canvas->root()));
i->view->property_data_src() = static_cast<gpointer>(i->region.get()); (*i)->view->property_data_src() = static_cast<gpointer>((*i)->region.get());
i->view->property_cache() = WaveView::create_cache (); (*i)->view->property_cache() = WaveView::create_cache ();
i->view->property_cache_updater() = true; (*i)->view->property_cache_updater() = true;
i->view->property_channel() = 0; (*i)->view->property_channel() = 0;
i->view->property_length_function() = (void *) region_length_from_c; (*i)->view->property_length_function() = (void *) region_length_from_c;
i->view->property_sourcefile_length_function() = (void *) sourcefile_length_from_c; (*i)->view->property_sourcefile_length_function() = (void *) sourcefile_length_from_c;
i->view->property_peak_function() = (void *) region_read_peaks_from_c; (*i)->view->property_peak_function() = (void *) region_read_peaks_from_c;
i->view->property_x() = 0; (*i)->view->property_x() = 0;
i->view->property_y() = n * _wave_height; (*i)->view->property_y() = n * _wave_height;
i->view->property_height() = _wave_height; (*i)->view->property_height() = _wave_height;
i->view->property_samples_per_unit() = i->samples_per_unit; (*i)->view->property_samples_per_unit() = (*i)->samples_per_unit;
i->view->property_region_start() = i->region->start(); (*i)->view->property_region_start() = (*i)->region->start();
i->view->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get(); (*i)->view->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
i->view->property_fill_color() = ARDOUR_UI::config()->canvasvar_WaveFormFill.get(); (*i)->view->property_fill_color() = ARDOUR_UI::config()->canvasvar_WaveFormFill.get();
(*i)->view->property_logscaled() = true;
(*i)->view->property_rectified() = true;
} }
++n; ++n;
@ -202,33 +218,34 @@ StripSilenceDialog::canvas_allocation (Gtk::Allocation& alloc)
_wave_width = alloc.get_width (); _wave_width = alloc.get_width ();
_wave_height = alloc.get_height (); _wave_height = alloc.get_height ();
for (std::list<Wave>::iterator i = _waves.begin(); i != _waves.end(); ++i, ++n) { for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i, ++n) {
i->samples_per_unit = ((double) i->region->length() / _wave_width); (*i)->samples_per_unit = ((double) (*i)->region->length() / _wave_width);
if (i->view) { if ((*i)->view) {
i->view->property_y() = n * _wave_height; (*i)->view->property_y() = n * _wave_height;
i->view->property_samples_per_unit() = i->samples_per_unit; (*i)->view->property_samples_per_unit() = (*i)->samples_per_unit;
i->view->property_height() = _wave_height; (*i)->view->property_height() = _wave_height;
} }
} }
redraw_silence_rects (); resize_silence_rects ();
update_threshold_line ();
} }
void void
StripSilenceDialog::redraw_silence_rects () StripSilenceDialog::resize_silence_rects ()
{ {
int n = 0; int n = 0;
for (std::list<Wave>::iterator i = _waves.begin(); i != _waves.end(); ++i) { for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
std::list<std::pair<frameoffset_t, framecnt_t> >::const_iterator j; list<pair<frameoffset_t, framecnt_t> >::const_iterator j;
std::list<SimpleRect*>::iterator r; list<SimpleRect*>::iterator r;
for (j = i->silence.begin(), r = i->silence_rects.begin(); for (j = (*i)->silence.begin(), r = (*i)->silence_rects.begin();
j != i->silence.end() && r != i->silence_rects.end(); ++j, ++r) { j != (*i)->silence.end() && r != (*i)->silence_rects.end(); ++j, ++r) {
(*r)->property_x1() = j->first / i->samples_per_unit; (*r)->property_x1() = j->first / (*i)->samples_per_unit;
(*r)->property_x2() = j->second / i->samples_per_unit; (*r)->property_x2() = j->second / (*i)->samples_per_unit;
(*r)->property_y1() = n * _wave_height; (*r)->property_y1() = n * _wave_height;
(*r)->property_y2() = (n + 1) * _wave_height; (*r)->property_y2() = (n + 1) * _wave_height;
(*r)->property_outline_pixels() = 0; (*r)->property_outline_pixels() = 0;
@ -239,6 +256,26 @@ StripSilenceDialog::redraw_silence_rects ()
} }
} }
void
StripSilenceDialog::update_threshold_line ()
{
int n = 0;
for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
(*i)->threshold_line->property_x1() = 0;
(*i)->threshold_line->property_x2() = _wave_width;
double const y = alt_log_meter (_threshold.get_value());
cout << "thresh " << _threshold.get_value() << " alt log " << alt_log_meter (_threshold.get_value()) << "\n";
(*i)->threshold_line->property_y1() = (n + 1 - y) * _wave_height;
(*i)->threshold_line->property_y2() = (n + 1 - y) * _wave_height;
}
++n;
}
void void
StripSilenceDialog::update_silence_rects () StripSilenceDialog::update_silence_rects ()
{ {
@ -246,24 +283,24 @@ StripSilenceDialog::update_silence_rects ()
uint32_t max_segments = 0; uint32_t max_segments = 0;
uint32_t sc; uint32_t sc;
for (std::list<Wave>::iterator i = _waves.begin(); i != _waves.end(); ++i) { for (list<Wave*>::iterator i = _waves.begin(); i != _waves.end(); ++i) {
for (std::list<SimpleRect*>::iterator j = i->silence_rects.begin(); j != i->silence_rects.end(); ++j) { for (list<SimpleRect*>::iterator j = (*i)->silence_rects.begin(); j != (*i)->silence_rects.end(); ++j) {
delete *j; delete *j;
} }
i->silence_rects.clear (); (*i)->silence_rects.clear ();
sc = 0; sc = 0;
for (std::list<std::pair<frameoffset_t, framecnt_t> >::const_iterator j = i->silence.begin(); j != i->silence.end(); ++j) { for (list<pair<frameoffset_t, framecnt_t> >::const_iterator j = (*i)->silence.begin(); j != (*i)->silence.end(); ++j) {
SimpleRect* r = new SimpleRect (*(_canvas->root())); SimpleRect* r = new SimpleRect (*(_canvas->root()));
r->property_x1() = j->first / i->samples_per_unit; r->property_x1() = j->first / (*i)->samples_per_unit;
r->property_x2() = j->second / i->samples_per_unit; r->property_x2() = j->second / (*i)->samples_per_unit;
r->property_y1() = n * _wave_height; r->property_y1() = n * _wave_height;
r->property_y2() = (n + 1) * _wave_height; r->property_y2() = (n + 1) * _wave_height;
r->property_outline_pixels() = 0; r->property_outline_pixels() = 0;
r->property_fill_color_rgba() = RGBA_TO_UINT (128, 128, 128, 128); r->property_fill_color_rgba() = RGBA_TO_UINT (128, 128, 128, 128);
i->silence_rects.push_back (r); (*i)->silence_rects.push_back (r);
sc++; sc++;
} }
@ -279,24 +316,32 @@ StripSilenceDialog::update_silence_rects ()
ma = (float) min_audible/_session->frame_rate(); ma = (float) min_audible/_session->frame_rate();
ms = (float) min_silence/_session->frame_rate(); ms = (float) min_silence/_session->frame_rate();
if (min_audible > _session->frame_rate()) { if (min_audible < _session->frame_rate()) {
aunits = _("secs"); aunits = _("ms");
ma /= 1000.0; ma *= 1000.0;
} else { } else {
aunits = _("msecs"); aunits = _("s");
} }
if (min_silence > _session->frame_rate()) { if (min_silence < _session->frame_rate()) {
sunits = _("secs"); sunits = _("ms");
ms /= 1000.0; ms *= 1000.0;
} else { } else {
sunits = _("msecs"); sunits = _("s");
} }
_segment_count_label.set_text (string_compose (_("Silent segments: %1\nShortest silence %2 %3 Shortest audible %4 %5"), _segment_count_label.set_text (string_compose ("%1", max_segments));
max_segments, ms, sunits, ma, aunits)); if (max_segments > 0) {
_shortest_silence_label.set_text (string_compose ("%1 %2", ms, sunits));
_shortest_audible_label.set_text (string_compose ("%1 %2", ma, aunits));
} else {
_shortest_silence_label.set_text ("");
_shortest_audible_label.set_text ("");
}
} else { } else {
_segment_count_label.set_text (_("Full silence")); _segment_count_label.set_text (_("Full silence"));
_shortest_silence_label.set_text ("");
_shortest_audible_label.set_text ("");
} }
} }
@ -343,9 +388,9 @@ StripSilenceDialog::detection_thread_work ()
StripSilenceDialog* ssd = current; StripSilenceDialog* ssd = current;
run_lock.unlock (); run_lock.unlock ();
for (std::list<Wave>::iterator i = ssd->_waves.begin(); i != ssd->_waves.end(); ++i) { for (list<Wave*>::iterator i = ssd->_waves.begin(); i != ssd->_waves.end(); ++i) {
i->silence = i->region->find_silence (dB_to_coefficient (ssd->threshold ()), ssd->minimum_length (), ssd->itt); (*i)->silence = (*i)->region->find_silence (dB_to_coefficient (ssd->threshold ()), ssd->minimum_length (), ssd->itt);
ssd->update_stats (i->silence); ssd->update_stats ((*i)->silence);
} }
if (!ssd->itt.cancel) { if (!ssd->itt.cancel) {
@ -360,6 +405,13 @@ StripSilenceDialog::detection_thread_work ()
return 0; return 0;
} }
void
StripSilenceDialog::threshold_changed ()
{
update_threshold_line ();
maybe_start_silence_detection ();
}
void void
StripSilenceDialog::maybe_start_silence_detection () StripSilenceDialog::maybe_start_silence_detection ()
{ {
@ -486,11 +538,27 @@ StripSilenceDialog::update_stats (const SilenceResult& res)
nframes_t nframes_t
StripSilenceDialog::minimum_length () const StripSilenceDialog::minimum_length () const
{ {
return _minimum_length.current_duration (_waves.front().region->position()); return _minimum_length.current_duration (_waves.front()->region->position());
} }
nframes_t nframes_t
StripSilenceDialog::fade_length () const StripSilenceDialog::fade_length () const
{ {
return _minimum_length.current_duration (_waves.front().region->position()); return _minimum_length.current_duration (_waves.front()->region->position());
}
StripSilenceDialog::Wave::Wave (Group* g, boost::shared_ptr<AudioRegion> r)
: region (r), view (0), samples_per_unit (1)
{
threshold_line = new ArdourCanvas::SimpleLine (*g);
threshold_line->property_color_rgba() = RGBA_TO_UINT (0, 0, 0, 128);
}
StripSilenceDialog::Wave::~Wave ()
{
delete view;
delete threshold_line;
for (list<SimpleRect*>::iterator i = silence_rects.begin(); i != silence_rects.end(); ++i) {
delete *i;
}
} }

View file

@ -49,26 +49,32 @@ private:
void peaks_ready (); void peaks_ready ();
void canvas_allocation (Gtk::Allocation &); void canvas_allocation (Gtk::Allocation &);
void update_silence_rects (); void update_silence_rects ();
void redraw_silence_rects (); void resize_silence_rects ();
void update_threshold_line ();
void threshold_changed ();
Gtk::SpinButton _threshold; Gtk::SpinButton _threshold;
AudioClock _minimum_length; AudioClock _minimum_length;
AudioClock _fade_length; AudioClock _fade_length;
Gtk::Label _segment_count_label; Gtk::Label _segment_count_label;
Gtk::Label _shortest_silence_label;
Gtk::Label _shortest_audible_label;
typedef std::list<std::pair<ARDOUR::frameoffset_t,ARDOUR::framecnt_t> > SilenceResult; typedef std::list<std::pair<ARDOUR::frameoffset_t,ARDOUR::framecnt_t> > SilenceResult;
struct Wave { struct Wave {
boost::shared_ptr<ARDOUR::AudioRegion> region; boost::shared_ptr<ARDOUR::AudioRegion> region;
ArdourCanvas::WaveView* view; ArdourCanvas::WaveView* view;
std::list<ArdourCanvas::SimpleRect*> silence_rects; std::list<ArdourCanvas::SimpleRect*> silence_rects;
ArdourCanvas::SimpleLine* threshold_line;
double samples_per_unit; double samples_per_unit;
SilenceResult silence; SilenceResult silence;
Wave() : view (0), samples_per_unit (1) { } Wave (ArdourCanvas::Group *, boost::shared_ptr<ARDOUR::AudioRegion>);
~Wave ();
}; };
ArdourCanvas::Canvas* _canvas; ArdourCanvas::Canvas* _canvas;
std::list<Wave> _waves; std::list<Wave*> _waves;
int _wave_width; int _wave_width;
int _wave_height; int _wave_height;
bool restart_queued; bool restart_queued;