comparison Test.cpp @ 61:c7fd03f5ae02

Show feature diffs

     }
     return false;
 }
 
 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 {
         cout << "    Values (" << f.values.size() << "): (elided)" << endl;
     }

                 Plugin::FeatureList::const_iterator afi = ai->second.begin();
                 Plugin::FeatureList::const_iterator bfi = bi->second.begin();
                 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;
                             }
                             if (afi->hasTimestamp && (afi->timestamp != bfi->timestamp)) {
                                 cout << "*** Feature " << fno << " differs in timestamp (" << afi->timestamp << " vs " << bfi->timestamp << " )" << endl;

                             if (afi->hasDuration && (afi->duration != bfi->duration)) {
                                 cout << "*** Feature " << fno << " differs in duration (" << afi->duration << " vs " << bfi->duration << " )" << endl;
                             }
                             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;
                     }
                     ++fno;
                     ++afi;