comparison data/model/SparseModel.h @ 777:eea8049df526

Fix incorrect retrieval of point in model by row number if point frame values fell between model resolution boundaries

     mutable std::vector<long> m_rows; // map from row number to frame
     void rebuildRowVector() const
     {
         m_rows.clear();
         for (PointListConstIterator i = m_points.begin(); i != m_points.end(); ++i) {
+//            std::cerr << "rebuildRowVector: row " << m_rows.size() << " -> " << i->frame << std::endl;
             m_rows.push_back(i->frame);
         }
     }
 
     PointListIterator getPointListIteratorForRow(int row)

         PointListIterator i0, i1;
         getPointIterators(frame, i0, i1);
         PointListIterator i = i0;
 
         for (i = i0; i != i1; ++i) {
+            if (i->frame < frame) { continue; }
             if (indexAtFrame > 0) { --indexAtFrame; continue; }
             return i;
         }
 
         if (indexAtFrame > 0) {

         int indexAtFrame = 0;
         int ri = row;
         while (ri > 0 && m_rows[ri-1] == m_rows[row]) { --ri; ++indexAtFrame; }
         int initialIndexAtFrame = indexAtFrame;
 
+//        std::cerr << "getPointListIteratorForRow " << row << ": initialIndexAtFrame = " << initialIndexAtFrame << std::endl;
+
         PointListConstIterator i0, i1;
         getPointIterators(frame, i0, i1);
         PointListConstIterator i = i0;
 
         for (i = i0; i != i1; ++i) {
+            if (i->frame < frame) { continue; }
             if (indexAtFrame > 0) { --indexAtFrame; continue; }
             return i;
         }
 
         if (indexAtFrame > 0) {

 
     long start = (frame / m_resolution) * m_resolution;
     long end = start + m_resolution;
 
     PointType startPoint(start), endPoint(end);
-    
+
     startItr = m_points.lower_bound(startPoint);
       endItr = m_points.upper_bound(endPoint);
 }
 
 template <typename PointType>

 
     long start = (frame / m_resolution) * m_resolution;
     long end = start + m_resolution;
 
     PointType startPoint(start), endPoint(end);
+    
+//    std::cerr << "getPointIterators: start frame " << start << ", end frame " << end << ", m_resolution " << m_resolution << std::endl;
     
     startItr = m_points.lower_bound(startPoint);
       endItr = m_points.upper_bound(endPoint);
 }
 

 void
 SparseModel<PointType>::toXml(QTextStream &out,
                               QString indent,
                               QString extraAttributes) const
 {
-    std::cerr << "SparseModel::toXml: extraAttributes = \"" 
-              << extraAttributes.toStdString() << std::endl;
+//    std::cerr << "SparseModel::toXml: extraAttributes = \"" 
+//              << extraAttributes.toStdString() << std::endl;
 
     QString type = getXmlOutputType();
 
     Model::toXml
 	(out,