comparison data/fileio/CSVFileReader.cpp @ 676:611a4fa14dde sv_v1.8

* Avoid including non-value columns in 3d-model column count (fixes #38)

     size_t startFrame = 0; // for calculation of dense model resolution
     bool firstEverValue = true;
 
     std::map<QString, int> labelCountMap;
     
+    int valueColumns = 0;
+    for (int i = 0; i < m_format.getColumnCount(); ++i) {
+        if (m_format.getColumnPurpose(i) == CSVFormat::ColumnValue) {
+            ++valueColumns;
+        }
+    }
+
     while (!in.atEnd()) {
 
         // QTextStream's readLine doesn't cope with old-style Mac
         // CR-only line endings.  Why did they bother making the class
         // cope with more than one sort of line ending, if it still

 		
                 case CSVFormat::ThreeDimensionalModel:
                     model3 = new EditableDenseThreeDimensionalModel
                         (sampleRate,
                          windowSize,
-                         list.size(),
+                         valueColumns,
                          EditableDenseThreeDimensionalModel::NoCompression);
                     model = model3;
                     break;
                 }
             }

 
                 DenseThreeDimensionalModel::Column values;
 
                 for (int i = 0; i < list.size(); ++i) {
 
+                    if (m_format.getColumnPurpose(i) != CSVFormat::ColumnValue) {
+                        continue;
+                    }
+
                     bool ok = false;
                     float value = list[i].toFloat(&ok);
 
-                    if (m_format.getColumnPurpose(i) == CSVFormat::ColumnValue) {
-                        values.push_back(value);
-                    }
+                    values.push_back(value);
 	    
                     if (firstEverValue || value < min) min = value;
                     if (firstEverValue || value > max) max = value;
-
+                    
                     if (firstEverValue) {
                         startFrame = frameNo;
                         model3->setStartFrame(startFrame);
                     } else if (lineno == 1 &&
                                timingType == CSVFormat::ExplicitTiming) {