Last @paulromano comment I held off on. Didnt update all mgxs h5 libraries yet as future comments from others will require I do that anyways.

openmc/mesh.py

@@ -302,8 +302,8 @@ class Mesh(object):
             # inconsistency between what numpy uses as the max float and what
             # Fortran expects for a real(8), so this avoids code complication
             # and achieves the same goal.
-            zplanes = [openmc.ZPlane(z0=-1e10., boundary_type='reflective'),
-                       openmc.ZPlane(z0=1e10., boundary_type='reflective')]
+            zplanes = [openmc.ZPlane(z0=-1e10, boundary_type='reflective'),
+                       openmc.ZPlane(z0=1e10, boundary_type='reflective')]
         else:
             zplanes = [openmc.ZPlane(z0=self.lower_left[2],
                                      boundary_type=bc[4]),

openmc/mgxs_library.py

@@ -306,7 +306,7 @@ class XSdata(object):
     def temperatures(self, temperatures):
         check_iterable_type('temperatures', temperatures, Real)
 
-        self._temperatures = temperatures
+        self._temperatures = np.array(temperatures)
 
     @scatter_format.setter
     def scatter_format(self, scatter_format):

scripts/openmc-update-mgxs

@@ -63,17 +63,15 @@ def parse_args():
 
 
 def get_data(element, entry):
-    try:
-        value = element.find(entry).text
-    except:
+    value = element.find(entry)
+    if value is not None:
+        value = value.text.strip()
+    else:
         if entry in element.attrib:
-            value = element.attrib[entry]
+            value = element.attrib[entry].strip()
         else:
             value = None
 
-    if value is not None:
-        value = value.strip()
-
     return value
 
 
@@ -140,7 +138,7 @@ if __name__ == '__main__':
         try:
             # It is in our list, so store that entry
             i = names.index(name)
-        except:
+        except ValueError:
             # It is not in our list, so add it
             i = -1
             xsd.append(openmc.XSdata(name, energy_groups,