Removing unused Block.coords argument rotationDegreesCCW (#1651)

armi/reactor/blocks.py

@@ -605,7 +605,7 @@ def getLocation(self):
         else:
             return "ExCore"
 
-    def coords(self, rotationDegreesCCW=0.0):
+    def coords(self):
         """
         Returns the coordinates of the block.
 
@@ -617,11 +617,7 @@ def coords(self, rotationDegreesCCW=0.0):
             method of the block's ``spatialLocator`` attribute, which recursively
             calls itself on all parents of the block to get the coordinates of the
             block's centroid in 3D cartesian space.
-
-            If ``rotationDegreesCCW`` is non-zero, an error is raised.
         """
-        if rotationDegreesCCW:
-            raise NotImplementedError("Cannot get coordinates with rotation.")
         return self.spatialLocator.getGlobalCoordinates()
 
     def setBuLimitInfo(self):
@@ -1677,7 +1673,7 @@ class HexBlock(Block):
     def __init__(self, name, height=1.0):
         Block.__init__(self, name, height)
 
-    def coords(self, rotationDegreesCCW=0.0):
+    def coords(self):
         """
         Returns the coordinates of the block.
 
@@ -1692,8 +1688,6 @@ def coords(self, rotationDegreesCCW=0.0):
 
             Will additionally adjust the x and y coordinates based on the block
             parameters ``displacementX`` and ``displacementY``.
-
-            Note that the ``rotationDegreesCCW`` argument is unused.
         """
         x, y, _z = self.spatialLocator.getGlobalCoordinates()
         x += self.p.displacementX * 100.0

doc/release/0.3.rst

@@ -13,6 +13,7 @@ What's new in ARMI?
 API Changes
 -----------
 #. Renaming ``structuredgrid.py`` to camelCase. (`PR#1650 <https://github.com/terrapower/armi/pull/1650>`_)
+#. Removing unused argument from ``Block.coords()``. (`PR#1651 <https://github.com/terrapower/armi/pull/1651>`_)
 #. TBD
 
 Bug Fixes
@@ -21,7 +22,7 @@ Bug Fixes
 
 Changes that Affect Requirements
 --------------------------------
-#. TBD
+#. (`PR#1651 <https://github.com/terrapower/armi/pull/1651>`_) - Very minor change to ``Block.coords()``, removing unused argument.
 
 
 ARMI v0.3.0