X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fdata%2Fsignalbase.cpp;h=9f47a9755eee6fc6ec085e3a831fb51062971183;hp=fe7c584b62b4eab7c58e2383b77989f0bd0194da;hb=63253d727162257aa0a3765bfb897c6826ebf611;hpb=f0f9c8566ba4992e3a3b71afd521da3278f37b7b diff --git a/pv/data/signalbase.cpp b/pv/data/signalbase.cpp index fe7c584b..9f47a975 100644 --- a/pv/data/signalbase.cpp +++ b/pv/data/signalbase.cpp @@ -26,11 +26,14 @@ #include "signalbase.hpp" #include "signaldata.hpp" +#include + #include #include using std::dynamic_pointer_cast; using std::make_shared; +using std::out_of_range; using std::shared_ptr; using std::tie; using std::unique_lock; @@ -40,6 +43,7 @@ namespace data { const int SignalBase::ColourBGAlpha = 8 * 256 / 100; const uint64_t SignalBase::ConversionBlockSize = 4096; +const uint32_t SignalBase::ConversionDelay = 1000; // 1 second SignalBase::SignalBase(shared_ptr channel, ChannelType channel_type) : channel_(channel), @@ -50,6 +54,11 @@ SignalBase::SignalBase(shared_ptr channel, ChannelType channel_ { if (channel_) internal_name_ = QString::fromStdString(channel_->name()); + + connect(&delayed_conversion_starter_, SIGNAL(timeout()), + this, SLOT(on_delayed_conversion_start())); + delayed_conversion_starter_.setSingleShot(true); + delayed_conversion_starter_.setInterval(ConversionDelay); } SignalBase::~SignalBase() @@ -338,7 +347,7 @@ SignalBase::ConversionPreset SignalBase::get_current_conversion_preset() const if (preset != conversion_options_.end()) return (ConversionPreset)((preset->second).toInt()); - return NoPreset; + return DynamicPreset; } void SignalBase::set_conversion_preset(ConversionPreset id) @@ -393,154 +402,188 @@ bool SignalBase::conversion_is_a2l() const (conversion_type_ == A2LConversionBySchmittTrigger))); } -void SignalBase::conversion_thread_proc(QObject* segment) +void SignalBase::convert_single_segment(AnalogSegment *asegment, LogicSegment *lsegment) { - // TODO Support for multiple segments is missing - uint64_t start_sample, end_sample; start_sample = end_sample = 0; - do { - if (conversion_is_a2l()) { + start_sample = lsegment->get_sample_count(); + end_sample = asegment->get_sample_count(); - AnalogSegment *asegment = qobject_cast(segment); + if (end_sample > start_sample) { + tie(min_value_, max_value_) = asegment->get_min_max(); - const shared_ptr logic_data = dynamic_pointer_cast(converted_data_); + // Create sigrok::Analog instance + float *asamples = new float[ConversionBlockSize]; + uint8_t *lsamples = new uint8_t[ConversionBlockSize]; - // Create the initial logic data segment if needed - if (logic_data->segments().size() == 0) { - shared_ptr lsegment = - make_shared(*logic_data.get(), 1, asegment->samplerate()); - logic_data->push_segment(lsegment); - } + vector > channels; + channels.push_back(channel_); + + vector mq_flags; + const sigrok::Quantity * const mq = sigrok::Quantity::VOLTAGE; + const sigrok::Unit * const unit = sigrok::Unit::VOLT; + + shared_ptr packet = + Session::sr_context->create_analog_packet(channels, + asamples, ConversionBlockSize, mq, unit, mq_flags); + + shared_ptr analog = + dynamic_pointer_cast(packet->payload()); + + // Convert + uint64_t i = start_sample; + + if (conversion_type_ == A2LConversionByThreshold) { + const double threshold = get_conversion_thresholds()[0]; - LogicSegment *lsegment = dynamic_cast(logic_data->segments().front().get()); + // Convert as many sample blocks as we can + while ((end_sample - i) > ConversionBlockSize) { + asegment->get_samples(i, i + ConversionBlockSize, asamples); - start_sample = lsegment->get_sample_count(); - end_sample = asegment->get_sample_count(); + shared_ptr logic = + analog->get_logic_via_threshold(threshold, lsamples); - if (end_sample > start_sample) { - tie(min_value_, max_value_) = asegment->get_min_max(); + lsegment->append_payload(logic->data_pointer(), logic->data_length()); - // Create sigrok::Analog instance - float *asamples = new float[ConversionBlockSize]; - uint8_t *lsamples = new uint8_t[ConversionBlockSize]; + samples_added(lsegment, i, i + ConversionBlockSize); + i += ConversionBlockSize; + } - vector > channels; - channels.push_back(channel_); + // Re-create sigrok::Analog and convert remaining samples + packet = Session::sr_context->create_analog_packet(channels, + asamples, end_sample - i, mq, unit, mq_flags); - vector mq_flags; - const sigrok::Quantity * const mq = sigrok::Quantity::VOLTAGE; - const sigrok::Unit * const unit = sigrok::Unit::VOLT; + analog = dynamic_pointer_cast(packet->payload()); - shared_ptr packet = - Session::sr_context->create_analog_packet(channels, - asamples, ConversionBlockSize, mq, unit, mq_flags); + asegment->get_samples(i, end_sample, asamples); + shared_ptr logic = + analog->get_logic_via_threshold(threshold, lsamples); + lsegment->append_payload(logic->data_pointer(), logic->data_length()); + samples_added(lsegment, i, end_sample); + } - shared_ptr analog = - dynamic_pointer_cast(packet->payload()); + if (conversion_type_ == A2LConversionBySchmittTrigger) { + const vector thresholds = get_conversion_thresholds(); + const double lo_thr = thresholds[0]; + const double hi_thr = thresholds[1]; - // Convert - uint64_t i = start_sample; + uint8_t state = 0; // TODO Use value of logic sample n-1 instead of 0 - if (conversion_type_ == A2LConversionByThreshold) { - const double threshold = get_conversion_thresholds()[0]; + // Convert as many sample blocks as we can + while ((end_sample - i) > ConversionBlockSize) { + asegment->get_samples(i, i + ConversionBlockSize, asamples); - // Convert as many sample blocks as we can - while ((end_sample - i) > ConversionBlockSize) { - asegment->get_samples(i, i + ConversionBlockSize, asamples); + shared_ptr logic = + analog->get_logic_via_schmitt_trigger(lo_thr, hi_thr, + &state, lsamples); - shared_ptr logic = - analog->get_logic_via_threshold(threshold, lsamples); + lsegment->append_payload(logic->data_pointer(), logic->data_length()); - lsegment->append_payload(logic->data_pointer(), logic->data_length()); + samples_added(lsegment, i, i + ConversionBlockSize); + i += ConversionBlockSize; + } - samples_added(lsegment, i, i + ConversionBlockSize); - i += ConversionBlockSize; - } + // Re-create sigrok::Analog and convert remaining samples + packet = Session::sr_context->create_analog_packet(channels, + asamples, end_sample - i, mq, unit, mq_flags); - // Re-create sigrok::Analog and convert remaining samples - packet = Session::sr_context->create_analog_packet(channels, - asamples, end_sample - i, mq, unit, mq_flags); + analog = dynamic_pointer_cast(packet->payload()); - analog = dynamic_pointer_cast(packet->payload()); + asegment->get_samples(i, end_sample, asamples); + shared_ptr logic = + analog->get_logic_via_schmitt_trigger(lo_thr, hi_thr, + &state, lsamples); + lsegment->append_payload(logic->data_pointer(), logic->data_length()); + samples_added(lsegment, i, end_sample); + } - asegment->get_samples(i, end_sample, asamples); - shared_ptr logic = - analog->get_logic_via_threshold(threshold, lsamples); - lsegment->append_payload(logic->data_pointer(), logic->data_length()); - samples_added(lsegment, i, end_sample); - } + // If acquisition is ongoing, start-/endsample may have changed + end_sample = asegment->get_sample_count(); - if (conversion_type_ == A2LConversionBySchmittTrigger) { - const vector thresholds = get_conversion_thresholds(); - const double lo_thr = thresholds[0]; - const double hi_thr = thresholds[1]; + delete[] lsamples; + delete[] asamples; + } +} - uint8_t state = 0; // TODO Use value of logic sample n-1 instead of 0 +void SignalBase::conversion_thread_proc() +{ + shared_ptr analog_data; - // Convert as many sample blocks as we can - while ((end_sample - i) > ConversionBlockSize) { - asegment->get_samples(i, i + ConversionBlockSize, asamples); + if (conversion_is_a2l()) { + analog_data = dynamic_pointer_cast(data_); - shared_ptr logic = - analog->get_logic_via_schmitt_trigger(lo_thr, hi_thr, - &state, lsamples); + if (analog_data->analog_segments().size() == 0) + return; - lsegment->append_payload(logic->data_pointer(), logic->data_length()); + } else + // Currently, we only handle A2L conversions + return; - samples_added(lsegment, i, i + ConversionBlockSize); - i += ConversionBlockSize; - } + uint32_t segment_id = 0; - // Re-create sigrok::Analog and convert remaining samples - packet = Session::sr_context->create_analog_packet(channels, - asamples, end_sample - i, mq, unit, mq_flags); + AnalogSegment *asegment = analog_data->analog_segments().front().get(); + assert(asegment); - analog = dynamic_pointer_cast(packet->payload()); + const shared_ptr logic_data = dynamic_pointer_cast(converted_data_); + assert(logic_data); - asegment->get_samples(i, end_sample, asamples); - shared_ptr logic = - analog->get_logic_via_schmitt_trigger(lo_thr, hi_thr, - &state, lsamples); - lsegment->append_payload(logic->data_pointer(), logic->data_length()); - samples_added(lsegment, i, end_sample); - } + // Create the initial logic data segment if needed + if (logic_data->logic_segments().size() == 0) { + shared_ptr new_segment = + make_shared(*logic_data.get(), 1, asegment->samplerate()); + logic_data->push_segment(new_segment); + } - // If acquisition is ongoing, start-/endsample may have changed - end_sample = asegment->get_sample_count(); + LogicSegment *lsegment = logic_data->logic_segments().front().get(); + assert(lsegment); - delete[] lsamples; - delete[] asamples; + do { + convert_single_segment(asegment, lsegment); + + if (analog_data->analog_segments().size() > logic_data->logic_segments().size()) { + // There are more segments to process + segment_id++; + + try { + asegment = analog_data->analog_segments().at(segment_id).get(); + } catch (out_of_range) { + qDebug() << "Conversion error for" << name() << ": no analog segment" \ + << segment_id << ", segments size is" << analog_data->analog_segments().size(); + return; } - } - if (!conversion_interrupt_ && (start_sample == end_sample)) { - unique_lock input_lock(conversion_input_mutex_); - conversion_input_cond_.wait(input_lock); + shared_ptr new_segment = + make_shared(*logic_data.get(), 1, asegment->samplerate()); + logic_data->push_segment(new_segment); + + lsegment = logic_data->logic_segments().back().get(); + } else { + // No more segments to process, wait for data or interrupt + if (!conversion_interrupt_) { + unique_lock input_lock(conversion_input_mutex_); + conversion_input_cond_.wait(input_lock); + } } } while (!conversion_interrupt_); } -void SignalBase::start_conversion() +void SignalBase::start_conversion(bool delayed_start) { + if (delayed_start) { + delayed_conversion_starter_.start(); + return; + } + stop_conversion(); if (converted_data_) converted_data_->clear(); + samples_cleared(); - if (conversion_is_a2l()) { - shared_ptr analog_data = dynamic_pointer_cast(data_); - - if (analog_data->analog_segments().size() > 0) { - // TODO Support for multiple segments is missing - AnalogSegment *asegment = analog_data->analog_segments().front().get(); - - conversion_interrupt_ = false; - conversion_thread_ = std::thread( - &SignalBase::conversion_thread_proc, this, asegment); - } - } + conversion_interrupt_ = false; + conversion_thread_ = std::thread( + &SignalBase::conversion_thread_proc, this); } void SignalBase::stop_conversion() @@ -568,8 +611,9 @@ void SignalBase::on_samples_added(QObject* segment, uint64_t start_sample, // Notify the conversion thread since it's running conversion_input_cond_.notify_one(); } else { - // Start the conversion thread - start_conversion(); + // Start the conversion thread unless the delay timer is running + if (!delayed_conversion_starter_.isActive()) + start_conversion(); } } @@ -578,13 +622,12 @@ void SignalBase::on_samples_added(QObject* segment, uint64_t start_sample, void SignalBase::on_min_max_changed(float min, float max) { - (void)min; - (void)max; - // Restart conversion if one is enabled and uses a calculated threshold if ((conversion_type_ != NoConversion) && (get_current_conversion_preset() == DynamicPreset)) - start_conversion(); + start_conversion(true); + + min_max_changed(min, max); } void SignalBase::on_capture_state_changed(int state) @@ -596,5 +639,10 @@ void SignalBase::on_capture_state_changed(int state) } } +void SignalBase::on_delayed_conversion_start() +{ + start_conversion(); +} + } // namespace data } // namespace pv