#include "evoral/EventSink.hpp"
#include "ardour/debug.h"
+#include "ardour/file_source.h"
+#include "ardour/midi_channel_filter.h"
#include "ardour/midi_model.h"
-#include "ardour/midi_state_tracker.h"
#include "ardour/midi_source.h"
-#include "ardour/file_source.h"
+#include "ardour/midi_state_tracker.h"
#include "ardour/session.h"
+#include "ardour/tempo.h"
#include "ardour/session_directory.h"
#include "ardour/source_factory.h"
-#include "i18n.h"
+#include "pbd/i18n.h"
namespace ARDOUR { template <typename T> class MidiRingBuffer; }
, _writing(false)
, _model_iter_valid(false)
, _length_beats(0.0)
+ , _length_pulse(0.0)
, _last_read_end(0)
, _capture_length(0)
, _capture_loop_length(0)
, _writing(false)
, _model_iter_valid(false)
, _length_beats(0.0)
+ , _length_pulse(0.0)
, _last_read_end(0)
, _capture_length(0)
, _capture_loop_length(0)
int
MidiSource::set_state (const XMLNode& node, int /*version*/)
{
- const XMLProperty* prop;
+ XMLProperty const * prop;
if ((prop = node.property ("captured-for")) != 0) {
_captured_for = prop->value();
}
}
void
-MidiSource::invalidate (const Lock& lock)
+MidiSource::invalidate (const Lock& lock, std::set<Evoral::Sequence<Evoral::Beats>::WeakNotePtr>* notes)
{
_model_iter_valid = false;
- _model_iter.invalidate();
+ _model_iter.invalidate(notes);
}
framecnt_t
framepos_t source_start,
framepos_t start,
framecnt_t cnt,
+ Evoral::Range<framepos_t>* loop_range,
MidiStateTracker* tracker,
- const std::set<Evoral::Parameter>& filtered) const
+ MidiChannelFilter* filter,
+ const std::set<Evoral::Parameter>& filtered,
+ const double pulse,
+ const double start_beats) const
{
- BeatsFramesConverter converter(_session.tempo_map(), source_start);
+ //BeatsFramesConverter converter(_session.tempo_map(), source_start);
+ const int32_t tpb = Timecode::BBT_Time::ticks_per_beat;
+ const double pulse_tick_res = floor ((pulse * 4.0 * tpb) + 0.5) / tpb;
+ const double start_qn = (pulse * 4.0) - start_beats;
DEBUG_TRACE (DEBUG::MidiSourceIO,
string_compose ("MidiSource::midi_read() %5 sstart %1 start %2 cnt %3 tracker %4\n",
source_start, start, cnt, tracker, name()));
- if (_model) {
- // Find appropriate model iterator
- Evoral::Sequence<Evoral::Beats>::const_iterator& i = _model_iter;
- if (_last_read_end == 0 || start != _last_read_end || !_model_iter_valid) {
- // Cached iterator is invalid, search for the first event past start
- i = _model->begin(converter.from(start), false, filtered);
- _model_iter_valid = true;
+ if (!_model) {
+ return read_unlocked (lm, dst, source_start, start, cnt, loop_range, tracker, filter);
+ }
+
+ // Find appropriate model iterator
+ Evoral::Sequence<Evoral::Beats>::const_iterator& i = _model_iter;
+ const bool linear_read = _last_read_end != 0 && start == _last_read_end;
+ if (!linear_read || !_model_iter_valid) {
+#if 0
+ // Cached iterator is invalid, search for the first event past start
+ i = _model->begin(converter.from(start), false, filtered,
+ linear_read ? &_model->active_notes() : NULL);
+ _model_iter_valid = true;
+ if (!linear_read) {
+ _model->active_notes().clear();
+ }
+#else
+ /* hot-fix http://tracker.ardour.org/view.php?id=6541
+ * "parallel playback of linked midi regions -> no note-offs"
+ *
+ * A midi source can be used by multiple tracks simultaneously,
+ * in which case midi_read() may be called from different tracks for
+ * overlapping time-ranges.
+ *
+ * However there is only a single iterator for a given midi-source.
+ * This results in every midi_read() performing a seek.
+ *
+ * If seeking is performed with
+ * _model->begin(converter.from(start),...)
+ * the model is used for seeking. That method seeks to the first
+ * *note-on* event after 'start'.
+ *
+ * _model->begin(converter.from( ) ,..) eventually calls
+ * Sequence<Time>::const_iterator() in libs/evoral/src/Sequence.cpp
+ * which looks up the note-event via seq.note_lower_bound(t);
+ * but the sequence 'seq' only contains note-on events(!).
+ * note-off events are implicit in Sequence<Time>::operator++()
+ * via _active_notes.pop(); and not part of seq.
+ *
+ * see also http://tracker.ardour.org/view.php?id=6287#c16671
+ *
+ * The linear search below assures that reading starts at the first
+ * event for the given time, regardless of its event-type.
+ *
+ * The performance of this approach is O(N), while the previous
+ * implementation is O(log(N)). This needs to be optimized:
+ * The model-iterator or event-sequence needs to be re-designed in
+ * some way (maybe keep an iterator per playlist).
+ */
+ for (i = _model->begin(); i != _model->end(); ++i) {
+ if (floor (((i->time().to_double() + start_qn) * tpb) + 0.5) / tpb >= pulse_tick_res) {
+ break;
+ }
+ }
+ _model_iter_valid = true;
+ if (!linear_read) {
+ _model->active_notes().clear();
}
+#endif
+ }
+
+ _last_read_end = start + cnt;
+
+ // Copy events in [start, start + cnt) into dst
+ for (; i != _model->end(); ++i) {
+
+ // Offset by source start to convert event time to session time
+
+ framecnt_t time_frames = _session.tempo_map().frame_at_quarter_note (i->time().to_double() + start_qn);
+
+ if (time_frames < (start + source_start)) {
- _last_read_end = start + cnt;
+ /* event too early */
+
+ continue;
+
+ } else if (time_frames >= start + cnt + source_start) {
+
+ DEBUG_TRACE (DEBUG::MidiSourceIO,
+ string_compose ("%1: reached end with event @ %2 vs. %3\n",
+ _name, time_frames, start+cnt));
+ break;
+
+ } else {
- // Copy events in [start, start + cnt) into dst
- for (; i != _model->end(); ++i) {
- const framecnt_t time_frames = converter.to(i->time());
- if (time_frames < start + cnt) {
- // Offset by source start to convert event time to session time
- dst.write (time_frames + source_start, i->event_type(), i->size(), i->buffer());
+ /* in range */
+ if (filter && filter->filter(i->buffer(), i->size())) {
DEBUG_TRACE (DEBUG::MidiSourceIO,
- string_compose ("%1: add event @ %2 type %3 size %4\n",
- _name, time_frames + source_start, i->event_type(), i->size()));
+ string_compose ("%1: filter event @ %2 type %3 size %4\n",
+ _name, time_frames, i->event_type(), i->size()));
+ continue;
+ }
+
+ if (loop_range) {
+ time_frames = loop_range->squish (time_frames);
+ }
- if (tracker) {
- tracker->track (*i);
+ dst.write (time_frames, i->event_type(), i->size(), i->buffer());
+
+#ifndef NDEBUG
+ if (DEBUG_ENABLED(DEBUG::MidiSourceIO)) {
+ DEBUG_STR_DECL(a);
+ DEBUG_STR_APPEND(a, string_compose ("%1 added event @ %2 sz %3 within %4 .. %5\n",
+ _name, time_frames, i->size(),
+ start + source_start, start + cnt + source_start));
+ for (size_t n=0; n < i->size(); ++n) {
+ DEBUG_STR_APPEND(a,hex);
+ DEBUG_STR_APPEND(a,"0x");
+ DEBUG_STR_APPEND(a,(int)i->buffer()[n]);
+ DEBUG_STR_APPEND(a,' ');
}
- } else {
- DEBUG_TRACE (DEBUG::MidiSourceIO,
- string_compose ("%1: reached end with event @ %2 vs. %3\n",
- _name, time_frames, start+cnt));
- break;
+ DEBUG_STR_APPEND(a,'\n');
+ DEBUG_TRACE (DEBUG::MidiSourceIO, DEBUG_STR(a).str());
+ }
+#endif
+
+ if (tracker) {
+ tracker->track (*i);
}
}
- return cnt;
- } else {
- return read_unlocked (lm, dst, source_start, start, cnt, tracker);
}
+
+ return cnt;
}
framecnt_t
_capture_length = capture_length;
_capture_loop_length = loop_length;
- BeatsFramesConverter converter(_session.tempo_map(), position);
+ TempoMap& map (_session.tempo_map());
+ BeatsFramesConverter converter(map, position);
_length_beats = converter.from(capture_length);
+ _length_pulse = map.pulse_at_frame (position + capture_length) - map.pulse_at_frame (position);
}
void
mark_midi_streaming_write_completed (lock, Evoral::Sequence<Evoral::Beats>::DeleteStuckNotes);
}
+int
+MidiSource::export_write_to (const Lock& lock, boost::shared_ptr<MidiSource> newsrc, Evoral::Beats begin, Evoral::Beats end)
+{
+ Lock newsrc_lock (newsrc->mutex ());
+
+ if (!_model) {
+ error << string_compose (_("programming error: %1"), X_("no model for MidiSource during export"));
+ return -1;
+ }
+
+ _model->write_section_to (newsrc, newsrc_lock, begin, end, true);
+
+ newsrc->flush_midi(newsrc_lock);
+
+ return 0;
+}
+
int
MidiSource::write_to (const Lock& lock, boost::shared_ptr<MidiSource> newsrc, Evoral::Beats begin, Evoral::Beats end)
{