Mercurial > hg > vamp-plugin-tester

changeset 61:c7fd03f5ae02

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Show feature diffs
author Chris Cannam
date Tue, 08 Sep 2015 11:28:42 +0100
parents 9f2a6d843639
children d19bbfc055ab
files Test.cpp Test.h
diffstat 2 files changed, 39 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/Test.cpp	Fri Sep 04 13:48:51 2015 +0100
+++ b/Test.cpp	Tue Sep 08 11:28:42 2015 +0100
@@ -192,15 +192,42 @@
 }
 
 void
-Test::dumpFeature(const Plugin::Feature &f, bool showValues)
+Test::dumpFeature(const Plugin::Feature &f, bool showValues,
+                  const Plugin::Feature *other)
 {
     cout << "    Timestamp: " << (!f.hasTimestamp ? "(none)" : f.timestamp.toText()) << endl;
     cout << "    Duration: " << (!f.hasDuration ? "(none)" : f.duration.toText()) << endl;
     cout << "    Label: " << (f.label == "" ? "(none)" : f.label) << endl;
     if (showValues) {
         cout << "    Values (" << f.values.size() << "): " << (f.values.empty() ? "(none)" : "");
-        for (int j = 0; j < (int)f.values.size(); ++j) {
-            cout << f.values[j] << " ";
+        int n = f.values.size();
+        if (!other) {
+            for (int j = 0; j < n; ++j) {
+                cout << f.values[j] << " ";
+            }
+        } else {
+            int samecount = 0;
+            int diffcount = 0;
+            for (int j = 0; j <= n; ++j) {
+                if (j < n && f.values[j] == other->values[j]) {
+                    ++samecount;
+                } else {
+                    if (samecount > 0) {
+                        cout << "(" << samecount << " identical) ";
+                    }
+                    samecount = 0;
+                    if (j < n) {
+                        ++diffcount;
+                        if (diffcount > 20 && j + 10 < n) {
+                            cout << "(remaining " << n - j << " values elided)";
+                            break;
+                        } else {
+                            cout << f.values[j] << " [diff "
+                                 << f.values[j] - other->values[j] << "] ";
+                        }
+                    }
+                }
+            }
         }
         cout << endl;
     } else {
@@ -271,7 +298,9 @@
                 while (afi != ai->second.end()) {
                     if (!(*afi == *bfi)) {
                         if (diffcount == 0) {
-                            bool differInValues = (afi->values != bfi->values);
+                            bool differInValues =
+                                (afi->values.size() == bfi->values.size() &&
+                                 afi->values != bfi->values);
                             if (afi->hasTimestamp != bfi->hasTimestamp) {
                                 cout << "*** Feature " << fno << " differs in presence of timestamp (" << afi->hasTimestamp << " vs " << bfi->hasTimestamp << ")" << endl;
                             }
@@ -287,13 +316,16 @@
                             if (afi->label != bfi->label) {
                                 cout << "*** Feature " << fno << " differs in label" << endl;
                             }
+                            if (afi->values.size() != bfi->values.size()) {
+                                cout << "*** Feature " << fno << " differs in number of values (" << afi->values.size() << " vs " << bfi->values.size() << ")" << endl;
+                            }
                             if (differInValues) {
                                 cout << "*** Feature " << fno << " differs in values" << endl;
                             }
                             cout << "  First output:" << endl;
                             dumpFeature(*afi, differInValues);
                             cout << "  Second output:" << endl;
-                            dumpFeature(*bfi, differInValues);
+                            dumpFeature(*bfi, differInValues, &(*afi));
                         }
                         ++diffcount;
                     }
--- a/Test.h	Fri Sep 04 13:48:51 2015 +0100
+++ b/Test.h	Tue Sep 08 11:28:42 2015 +0100
@@ -112,7 +112,8 @@
 
     bool containsTimestamps(const Vamp::Plugin::FeatureSet &);
 
-    void dumpFeature(const Vamp::Plugin::Feature &, bool showValues);
+    void dumpFeature(const Vamp::Plugin::Feature &, bool showValues,
+                     const Vamp::Plugin::Feature *other = 0);
     void dump(const Vamp::Plugin::FeatureSet &, bool showValues = true);
     void dumpTwo(const Result &r,
                  const Vamp::Plugin::FeatureSet &,