Cleanup before merge

models/cutouts.py

@@ -349,7 +349,6 @@ def save(self, filename=None, overwrite=True, **kwargs):
         # make sure to also save using the FileOnDiskMixin method
         FileOnDiskMixin.save(self, fullname, overwrite=overwrite, **kwargs)
 
-    # WHPR this might be able to become a class method... Is that better?
     def _load_dataset_dict_from_hdf5(self, file, groupname):
         """Load the dataset from an HDF5 group into one co_subdict and return.
 
@@ -404,7 +403,7 @@ def load(self, filepath=None):
         self.co_dict = Co_Dict()
         self.co_dict.cutouts = self
 
-        if os.path.exists(filepath): # WHPR revisit this check... necessary?
+        if os.path.exists(filepath):
             if self.format == 'hdf5':
                 with h5py.File(filepath, 'r') as file:
                     # quirk: the resulting dict is sorted alphabetically... likely harmless

models/measurements.py

@@ -396,8 +396,7 @@ def __setattr__(self, key, value):
             value = np.array(value)
 
         if key in ['center_x_pixel', 'center_y_pixel'] and value is not None:
-            value = int(round(value)) # improc/tools::make_cutouts uses np.round()
-                                      # should I change to match?
+            value = int(np.round(value))
 
         super().__setattr__(key, value)