Don't sub-class from object ... superfluous'

cta-observatory · Jul 8, 2015 · 8d5ecd6 · 8d5ecd6
1 parent dcf332b
commit 8d5ecd6
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/ctapipe/io/obsconfig.py b/ctapipe/io/obsconfig.py
@@ -29,7 +29,7 @@ def get_array_id_for_time( obstime ):
     return array_id, array_version
 
 
-class BaseConfig(object):
+class BaseConfig:
     pass
 
 

diff --git a/ctapipe/utils/fitshistogram.py b/ctapipe/utils/fitshistogram.py
@@ -8,7 +8,7 @@
 __all__ = ['Histogram']
 
 
-class Histogram(object):
+class Histogram:
     """A simple N-dimensional histogram class that can be written or read
     from a FITS file. 
 

diff --git a/ctapipe/visualization/mpl.py b/ctapipe/visualization/mpl.py
@@ -14,9 +14,8 @@
 logger = logging.getLogger(__name__)
 
 
-class CameraDisplay(object):
-
-    """Camera Display using MatPlotLib
+class CameraDisplay:
+    """Camera Display using matplotlib.
 
     Parameters
     ----------

diff --git a/docs/development/code-guidelines.rst b/docs/development/code-guidelines.rst
@@ -30,3 +30,6 @@ basic guidelines:
   added or removed, and the names are fixed.  Unlike C/C++ structs,
   namedtuples are _immutable_, meaning their data cannot be changed
   after they are constructed. 
+
+* Classes don't need to sub-class `object`, because we only support Python 3
+  and new-style classes are the default, i.e. subclassing `object` is superfluous.