Merge pull request #527 from SpiNNakerManchester/pylint_fixes

Minor fixes to get rid of pylint spelling cautions

.pylint_dict.txt

@@ -64,3 +64,4 @@ precompression
 Navaridas
 obj
 subobj
+xy

pacman/model/graphs/common/mdslice.py

@@ -76,7 +76,7 @@ def as_slice(self):
     @overrides(Slice.get_slice, extend_doc=False)
     def get_slice(self, n):
         """
-        Get a slice in the `n`'th dimension
+        Get a slice in the `n`'Th dimension
 
         :param int n: The 0-indexed dimension to get the shape of
         :type: slice

pacman/model/graphs/common/slice.py

@@ -117,7 +117,7 @@ def as_slice(self):
 
     def get_slice(self, n):
         """
-        Get a slice in the `n`'th dimension.
+        Get a slice in the `n`'Th dimension.
 
         :param int n: Must be 0
         :type: slice

pacman/model/resources/variable_sdram.py

@@ -35,7 +35,7 @@ def __init__(self, fixed_sdram, per_timestep_sdram):
         """
         :param fixed_sdram:
             The amount of SDRAM (in bytes) that represents static overhead
-        :type dtcm: int or numpy.integer
+        :type fixed_sdram: int or numpy.integer
         :param per_timestep_sdram:
             The amount of SDRAM (in bytes) required per timestep.
             Often represents the space to record a timestep.

pacman/model/routing_tables/abstract_multicast_routing_table.py

@@ -74,4 +74,4 @@ def __hash__(self):
 
     @abstractmethod
     def __repr__(self):
-        """repr"""
+        """ String representation of the table """

pacman/operations/router_algorithms/application_router.py

@@ -808,14 +808,14 @@ def _path_without_loops(start_xy, nodes):
     return nodes
 
 
-def _is_ok(coord, node, machine):
+def _is_ok(xy, node, machine):
     """
-    :param tuple(int, int) coord:
+    :param tuple(int, int) xy:
     :param tuple(int,tuple(int, int)) node:
     :param ~spinn_machine.Machine machine:
     :rtype: bool
     """
-    c_x, c_y = coord
+    c_x, c_y = xy
     direction, (n_x, n_y) = node
     if machine.is_link_at(c_x, c_y, direction):
         if machine.is_chip_at(n_x, n_y):

pacman/operations/router_algorithms/basic_dijkstra_routing.py

@@ -34,11 +34,6 @@
 
 
 class _NodeInfo(object):
-    """
-    :ivar list(~spinn_machine.Link) neighbours:
-    :ivar list(float) bws:
-    :ivar list(float) weights:
-    """
     __slots__ = ["neighbours", "bws", "weights"]
 
     def __init__(self):

pacman/operations/router_compressors/entry.py

@@ -40,16 +40,16 @@ def __eq__(self, other):
                 self.spinnaker_route == other.spinnaker_route)
 
     @staticmethod
-    def from_MulticastRoutingEntry(mre):
+    def from_MulticastRoutingEntry(source):
         """
-        :param ~spinn_machine.MulticastRoutingEntry mre:
+        :param ~spinn_machine.MulticastRoutingEntry source:
         :rtype: Entry
         """
         # Yes I know using _params is ugly but this is for speed
         # pylint:disable=protected-access
         return Entry(
-            mre._routing_entry_key, mre._mask, mre._defaultable,
-            mre._spinnaker_route)
+            source._routing_entry_key, source._mask, source._defaultable,
+            source._spinnaker_route)
 
     def to_MulticastRoutingEntry(self):
         """

pacman/utilities/utility_objs/__init__.py

@@ -12,7 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-from .field import Field, SUPPORTED_TAGS
 from .chip_counter import ChipCounter
 
-__all__ = ["Field", "SUPPORTED_TAGS", "ChipCounter"]
+__all__ = ["ChipCounter"]