Merge pull request #65 from Galvant/py3_support_work

Py3 support work

.travis.yml

@@ -10,8 +10,10 @@ install:
   - pip install python-coveralls
   - pip install coverage
   - pip install nose
+  - pip install pylint
 # command to run tests
 script:
   - nosetests --with-coverage -w instruments
+  - cd instruments && pylint --py3k instruments/
 after_success:
   - coveralls

instruments/instruments/__init__.py

@@ -23,6 +23,8 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 ##
 
+from __future__ import absolute_import
+
 ## VERSION METADATA ###########################################################
 # In keeping with PEP-396, we define a version number of the form
 # {major}.{minor}[.{postrelease}]{prerelease-tag}
@@ -40,7 +42,7 @@ def __check_versions():
         'flufl.enum': StrictVersion('4.0')
     }
 
-    for module_name, version in VERSIONS_NEEDED.iteritems():
+    for module_name, version in VERSIONS_NEEDED.items():
         module = __import__(module_name, fromlist=['__version__'])
         if StrictVersion(module.__version__) < version:
             raise ImportError("Module {} is version {}, but we need version {}.".format(module_name, module.__version__, version))

instruments/instruments/abstract_instruments/__init__.py

@@ -1,3 +1,8 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+from __future__ import absolute_import
+
 from instruments.abstract_instruments.instrument import Instrument
 from instruments.abstract_instruments.multimeter import Multimeter
 from instruments.abstract_instruments.electrometer import Electrometer

instruments/instruments/abstract_instruments/comm/__init__.py

@@ -1,3 +1,8 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+from __future__ import absolute_import
+
 from instruments.abstract_instruments.comm.abstract_comm import AbstractCommunicator
 
 from instruments.abstract_instruments.comm.socket_communicator import SocketCommunicator

instruments/instruments/abstract_instruments/comm/abstract_comm.py

@@ -22,19 +22,18 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 ##
 
-## FEATURES ###################################################################
+## IMPORTS ####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS ####################################################################
+from future.utils import with_metaclass
 
 import abc
 import logging
 
 ## CLASSES ####################################################################
 
-class AbstractCommunicator(object):
-    __metaclass__ = abc.ABCMeta
+class AbstractCommunicator(with_metaclass(abc.ABCMeta, object)):
 
     ## INITIALIZER ##
 

instruments/instruments/abstract_instruments/comm/file_communicator.py

@@ -25,6 +25,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import errno
 import io
 import time

instruments/instruments/abstract_instruments/comm/gi_gpib_communicator.py

@@ -25,6 +25,10 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+from builtins import chr
+
 import io
 import time
 
@@ -165,7 +169,7 @@ def eos(self, newval):
             self._eos = newval
         elif self._version >= 5:
             if isinstance(newval, int):
-                newval = str(unichr(newval))
+                newval = str(chr(newval))
             if newval == "\r\n":
                 self._eos = newval
                 newval = 0

instruments/instruments/abstract_instruments/comm/loopback_communicator.py

@@ -26,6 +26,11 @@
 
 # IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+from __future__ import print_function
+from builtins import input
+
 import io
 from instruments.abstract_instruments.comm import AbstractCommunicator
 import sys
@@ -110,14 +115,14 @@ def read(self, size=-1):
             else:
                 raise ValueError('Must read a positive value of characters.')
         else:
-            input_var = raw_input("Desired Response: ")
+            input_var = input("Desired Response: ")
         return input_var
 
     def write(self, msg):
         if self._stdout is not None:
             self._stdout.write(msg)
         else:
-            print " <- {} ".format(repr(msg))
+            print(" <- {} ".format(repr(msg)))
 
     def seek(self, offset):
         return NotImplemented

instruments/instruments/abstract_instruments/comm/serial_communicator.py

@@ -24,6 +24,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import io
 import serial
 

instruments/instruments/abstract_instruments/comm/serial_manager.py

@@ -34,6 +34,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import serial
 
 from instruments.abstract_instruments.comm import SerialCommunicator

instruments/instruments/abstract_instruments/comm/socket_communicator.py

@@ -25,6 +25,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import io
 import socket
 

instruments/instruments/abstract_instruments/comm/usb_communicator.py

@@ -24,6 +24,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import io
 
 import numpy as np

instruments/instruments/abstract_instruments/comm/usbtmc_communicator.py

@@ -25,6 +25,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import io
 
 from instruments.abstract_instruments.comm.abstract_comm import AbstractCommunicator

instruments/instruments/abstract_instruments/comm/visa_communicator.py

@@ -25,6 +25,9 @@
 
 ## IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import io
 
 # Trick to conditionally ignore the NameError caused by catching WindowsError.

instruments/instruments/abstract_instruments/electrometer.py

@@ -23,20 +23,19 @@
 ##
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
 from instruments.abstract_instruments import Instrument
 
 ## CLASSES #####################################################################
 
-class Electrometer(Instrument):
-    __metaclass__ = abc.ABCMeta
+class Electrometer(with_metaclass(abc.ABCMeta, Instrument)):
 
     ## PROPERTIES ##
 

instruments/instruments/abstract_instruments/function_generator.py

@@ -23,11 +23,11 @@
 ##
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
@@ -41,8 +41,7 @@
 
 ## CLASSES #####################################################################
 
-class FunctionGenerator(Instrument):
-    __metaclass__ = abc.ABCMeta
+class FunctionGenerator(with_metaclass(abc.ABCMeta, Instrument)):
 
     ## ENUMS ##
 

instruments/instruments/abstract_instruments/instrument.py

@@ -25,6 +25,9 @@
 
 # IMPORTS #####################################################################
 
+from __future__ import absolute_import
+from __future__ import division
+
 import socket
 import urlparse
 

instruments/instruments/abstract_instruments/multimeter.py

@@ -23,20 +23,19 @@
 ##
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
 from instruments.abstract_instruments import Instrument
 
 ## CLASSES #####################################################################
 
-class Multimeter(Instrument):
-    __metaclass__ = abc.ABCMeta
+class Multimeter(with_metaclass(abc.ABCMeta, Instrument)):
 
     ## PROPERTIES ##
 

instruments/instruments/abstract_instruments/oscilloscope.py

@@ -23,20 +23,19 @@
 ##
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
 from instruments.abstract_instruments import Instrument
 
 ## CLASSES #####################################################################
 
-class OscilloscopeDataSource(object):
-    __metaclass__ = abc.ABCMeta
+class OscilloscopeDataSource(with_metaclass(abc.ABCMeta, object)):
 
     ## PROPERTIES ##
 
@@ -50,8 +49,7 @@ def name(self):
     def read_waveform(self, bin_format=True):
         raise NotImplementedError
 
-class OscilloscopeChannel(object):
-    __metaclass__ = abc.ABCMeta
+class OscilloscopeChannel(with_metaclass(abc.ABCMeta, object)):
 
     ## PROPERTIES ##
 
@@ -68,8 +66,7 @@ def setcoupling(self, newval):
     coupling = abc.abstractproperty(getcoupling, setcoupling)
 
 
-class Oscilloscope(Instrument):
-    __metaclass__ = abc.ABCMeta
+class Oscilloscope(with_metaclass(abc.ABCMeta, Instrument)):
 
     ## PROPERTIES ##
 

instruments/instruments/abstract_instruments/power_supply.py

@@ -22,20 +22,19 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
 from instruments.abstract_instruments import Instrument
 
 ## CLASSES #####################################################################
 
-class PowerSupplyChannel(object):
-    __metaclass__ = abc.ABCMeta
+class PowerSupplyChannel(with_metaclass(abc.ABCMeta, object)):
 
     ## PROPERTIES ##
 
@@ -76,8 +75,7 @@ def setoutput(self, newval):
     output = abc.abstractproperty(getoutput, setoutput)
 
 
-class PowerSupply(Instrument):
-    __metaclass__ = abc.ABCMeta
+class PowerSupply(with_metaclass(abc.ABCMeta, Instrument)):
 
     ## PROPERTIES ##
 

instruments/instruments/abstract_instruments/signal_generator/__init__.py

@@ -1,3 +1,8 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+from __future__ import absolute_import
+
 from instruments.abstract_instruments.signal_generator.signal_generator import SignalGenerator
 from instruments.abstract_instruments.signal_generator.single_channel_sg import SingleChannelSG
 from instruments.abstract_instruments.signal_generator.channel import SGChannel

instruments/instruments/abstract_instruments/signal_generator/channel.py

@@ -22,27 +22,24 @@
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 ##
 
-## FEATURES ####################################################################
+## IMPORTS #####################################################################
 
+from __future__ import absolute_import
 from __future__ import division
-
-## IMPORTS #####################################################################
+from future.utils import with_metaclass
 
 import abc
 
-from instruments.abstract_instruments import Instrument
-
 ## CLASSES #####################################################################
 
-class SGChannel(object):
+class SGChannel(with_metaclass(abc.ABCMeta, object)):
     """
     Python abstract base class representing a single channel for a signal 
     generator. 
     
     .. warning:: This class should NOT be manually created by the user. It is 
         designed to be initialized by the `~instruments.SignalGenerator` class.
     """
-    __metaclass__ = abc.ABCMeta
 
     ## PROPERTIES ##