more comments

armi/materials/material.py

@@ -152,7 +152,7 @@ def getChildren(
         """Return empty list, representing that materials have no children."""
         return []
 
-    def getGrandparent(self):
+    def getAssociatedBlock(self):
         """Return the grandparent (typically block) associated with this material"""
         # would be nice to use composite.getAncestor, but material is not a composite
         component = self.parent
@@ -868,15 +868,15 @@ def __init__(self):
     def puFrac(self):
         """Get the Pu Frac."""
         # ideally the parameters would be stored on component, not Block (parent, not grandparent)
-        myBlock = self.getGrandparent()
+        myBlock = self.getAssociatedBlock()
         if myBlock is not None:
             return myBlock.p.puFrac
         runLog.warning(
             "{self} has no block attached, deferring to internally stored Pu Frac. "
             "Storing properties on the material objected will be deprecated in the near future.",
             single=true,
         )
-        return self_puFrac
+        return self._puFrac
 
     @puFrac.setter
     def puFrac(self, puFrac):