comparison rdf/RDFImporter.cpp @ 673:a1ae2c1f80ab

* Use note model when importing RDF types with "note" in them

              .arg(m_uristring));
         
         results = query.execute();
     }
 
-    for (int i = 0; i < results.size(); ++i) {
+    for (int i = 0; i < (int)results.size(); ++i) {
 
         QString signal = results[i]["signal"].value;
         QString source = results[i]["source"].value;
 
         std::cerr << "NOTE: Seeking signal source \"" << source.toStdString()

     if (query.wasCancelled()) {
         m_errorString = "Query cancelled";
         return;
     }        
 
-    for (int i = 0; i < results.size(); ++i) {
+    for (int i = 0; i < (int)results.size(); ++i) {
 
         QString feature = results[i]["feature"].value;
         QString type = results[i]["feature_signal_type"].value;
         QString value = results[i]["value"].value;
 

     // Map from timeline uri to event type to dimensionality to
     // presence of duration to model ptr.  Whee!
     std::map<QString, std::map<QString, std::map<int, std::map<bool, Model *> > > >
         modelMap;
 
-    for (int i = 0; i < results.size(); ++i) {
+    for (int i = 0; i < (int)results.size(); ++i) {
 
         if (i % 4 == 0) {
             if (reporter) reporter->setProgress(i/4);
         }
 

         bool haveTime = false;
         bool haveDuration = false;
 
         QString label = "";
         bool text = (type.contains("Text") || type.contains("text")); // Ha, ha
+        bool note = (type.contains("Note") || type.contains("note")); // Guffaw
 
         if (text) {
             label = SimpleSPARQLQuery::singleResultQuery
                 (s, textQueryString.arg(thinguri), "label").value;
         }

                     model = new NoteModel(m_sampleRate, 1, false);
                 }
 
             } else { // haveDuration
 
-                if (dimensions == 1 || dimensions == 2) {
+                if (note || (dimensions > 2)) {
+
+                    model = new NoteModel(m_sampleRate, 1, false);
+
+                } else {
 
                     // If our units are frequency or midi pitch, we
                     // should be using a note model... hm
                     
                     model = new RegionModel(m_sampleRate, 1, false);
-
-                } else {
-
-                    // We don't have a three-dimensional sparse model,
-                    // so use a note model.  We do have some logic (in
-                    // extractStructure below) for guessing whether
-                    // this should after all have been a dense model,
-                    // but it's hard to apply it because we don't have
-                    // all the necessary timing data yet... hmm
-
-                    model = new NoteModel(m_sampleRate, 1, false);
                 }
             }
 
             model->setRDFTypeURI(type);