# HG changeset patch
# User cannam
# Date 1218039482 0
# Node ID a5ede8515893103fd1a62e94fbb88c3b15eb4fd6
# Parent  2258794251bebdad9c507bf6b08dac15439588fb
* note in comment

diff -r 2258794251be -r a5ede8515893 vamp-sdk/hostext/PluginSummarisingAdapter.cpp
--- a/vamp-sdk/hostext/PluginSummarisingAdapter.cpp	Wed Aug 06 14:45:47 2008 +0000
+++ b/vamp-sdk/hostext/PluginSummarisingAdapter.cpp	Wed Aug 06 16:18:02 2008 +0000
@@ -268,6 +268,21 @@
 //!!! use timestamp to determine which segment we're on
     m_accumulators[output].count++;
     for (int i = 0; i < int(f.values.size()); ++i) {
+
+
+        //!!! we really want to associate this occurrence of this
+        //!!! value with the duration it covers.
+        
+        //!!! for dense values, the duration can be 1 or the sample
+        //!!! rate or whatever -- doesn't matter so long as it's the
+        //!!! same for every value.
+
+        //!!! for sparse values, the duration should be that from this
+        //!!! feature to the next.
+
+        //!!! if the feature has a duration, should be using that
+        //!!! instead.
+
         m_accumulators[output].values[i].push_back(f.values[i]);
     }
 }