Changes in / [4abcc93a:a4deca6] in sasview

Files:

• src/sas/sascalc/dataloader/readers/cansas_reader.py (modified) (8 diffs)
• test/sasdataloader/test/utest_abs_reader.py (modified) (7 diffs)

src/sas/sascalc/dataloader/readers/cansas_reader.py

@@ -206 +206 @@
         # Go through each child in the parent element
         for node in dom:
-            attr = node.attrib
-            name = attr.get("name", "")
-            type = attr.get("type", "")
             # Get the element name and set the current names level
             tagname = node.tag.replace(self.base_ns, "")
@@ -226 +223 @@
                 ## Recursion step to access data within the group
                 self._parse_entry(node)
-                if tagname == "SASsample":
-                    self.current_datainfo.sample.name = name
-                elif tagname == "beam_size":
-                    self.current_datainfo.source.beam_size_name = name
-                elif tagname == "SAScollimation":
-                    self.collimation.name = name
-                elif tagname == "aperture":
-                    self.aperture.name = name
-                    self.aperture.type = type
                 self.add_intermediate()
             else:
@@ -241 +229 @@
                 ## If this is a dataset, store the data appropriately
                 if tagname == 'Run':
-                    self.current_datainfo.run_name[data_point] = name
                     self.current_datainfo.run.append(data_point)
                 elif tagname == 'Title':
@@ -382 +369 @@
                     self.process.notes.append(data_point)
                 elif tagname == 'term' and self.parent_class == 'SASprocess':
-                    unit = attr.get("unit", "")
-                    dic = {}
-                    dic["name"] = name
-                    dic["value"] = data_point
-                    dic["unit"] = unit
-                    self.process.term.append(dic)
+                    self.process.term.append(data_point)
 
                 ## Transmission Spectrum
@@ -549 +531 @@
             self.current_datainfo.collimation.append(self.collimation)
             self.collimation = Collimation()
-        elif self.parent_class == 'aperture':
+        elif self.parent_class == 'SASaperture':
             self.collimation.aperture.append(self.aperture)
             self.aperture = Aperture()
@@ -664 +646 @@
         err_msg = None
         default_unit = None
-        if not isinstance(node_value, float):
-            node_value = float(node_value)
-        if 'unit' in attr and attr.get('unit') is not None:
+        if 'unit' in attr and attr.get('unit') is not None and not self.ns_list.ns_optional:
             try:
                 local_unit = attr['unit']
+                if not isinstance(node_value, float):
+                    node_value = float(node_value)
                 unitname = self.ns_list.current_level.get("unit", "")
                 if "SASdetector" in self.names:
@@ -925 +907 @@
                 self.write_node(point, "I", datainfo.y[i],
                                 {'unit': datainfo.y_unit})
-            if datainfo.dy is not None and len(datainfo.dy) > i:
+            if datainfo.dy != None and len(datainfo.dy) > i:
                 self.write_node(point, "Idev", datainfo.dy[i],
                                 {'unit': datainfo.y_unit})
-            if datainfo.dx is not None and len(datainfo.dx) > i:
+            if datainfo.dx != None and len(datainfo.dx) > i:
                 self.write_node(point, "Qdev", datainfo.dx[i],
                                 {'unit': datainfo.x_unit})
-            if datainfo.dxw is not None and len(datainfo.dxw) > i:
+            if datainfo.dxw != None and len(datainfo.dxw) > i:
                 self.write_node(point, "dQw", datainfo.dxw[i],
                                 {'unit': datainfo.x_unit})
-            if datainfo.dxl is not None and len(datainfo.dxl) > i:
+            if datainfo.dxl != None and len(datainfo.dxl) > i:
                 self.write_node(point, "dQl", datainfo.dxl[i],
                                 {'unit': datainfo.x_unit})
@@ -1202 +1184 @@
                 if isinstance(term, list):
                     value = term['value']
-                    del term['value']
-                elif isinstance(term, dict):
-                    value = term.get("value")
                     del term['value']
                 else:

test/sasdataloader/test/utest_abs_reader.py

@@ -176 +176 @@
         
         self.assertEqual(self.data.run[0], "1234")
-        self.assertEqual(self.data.meta_data['loader'], "CanSAS XML 1D")
+        self.assertEqual(self.data.meta_data['loader'], "CanSAS 1D")
+        self.assertEqual(len(self.data.errors), 0)
         
         # Data
@@ -197 +198 @@
         self.assertEqual(self.data.sample.name, "my sample")
         self.assertEqual(self.data.sample.thickness_unit, 'mm')
-        self.assertAlmostEqual(self.data.sample.thickness, 1.03)
-        
-        self.assertAlmostEqual(self.data.sample.transmission, 0.327)
+        self.assertEqual(self.data.sample.thickness, 1.03)
+        
+        self.assertEqual(self.data.sample.transmission, 0.327)
         
         self.assertEqual(self.data.sample.temperature_unit, 'C')
@@ -232 +233 @@
         
         self.assertEqual(self.data.source.wavelength_max_unit, "nm")
-        self.assertAlmostEqual(self.data.source.wavelength_max, 1.0)
+        self.assertEqual(self.data.source.wavelength_max, 1.0)
         self.assertEqual(self.data.source.wavelength_min_unit, "nm")
-        self.assertAlmostEqual(self.data.source.wavelength_min, 0.22)
+        self.assertEqual(self.data.source.wavelength_min, 0.22)
         self.assertEqual(self.data.source.wavelength_spread_unit, "percent")
         self.assertEqual(self.data.source.wavelength_spread, 14.3)
@@ -247 +248 @@
             self.assertEqual(item.size_unit,'mm')
             self.assertEqual(item.distance_unit,'mm')
-
+            
             if item.size.x==50 \
                 and item.distance==11000.0 \
@@ -293 +294 @@
             self.assertTrue(item.date in ['04-Sep-2007 18:35:02',
                                           '03-SEP-2006 11:42:47'])
-            print item.term
             for t in item.term:
                 if t['name']=="ABS:DSTAND" \
@@ -345 +345 @@
         self.assertEqual(self.data.filename, filename)
         # The followed should not have been loaded
-        self.assertAlmostEqual(self.data.sample.thickness, 0.00103)
+        self.assertEqual(self.data.sample.thickness, 0.00103)
         # This one should
-        self.assertAlmostEqual(self.data.sample.transmission, 0.327)
-        
-        self.assertEqual(self.data.meta_data['loader'], "CanSAS XML 1D")
+        self.assertEqual(self.data.sample.transmission, 0.327)
+        
+        self.assertEqual(self.data.meta_data['loader'], "CanSAS 1D")
         print self.data.errors
         self.assertEqual(len(self.data.errors), 1)
@@ -385 +385 @@
 
 if __name__ == '__main__':
-    unittest.main()
+    unittest.main(testRunner=unittest.TextTestRunner(verbosity=2))