Merge 71b18b6 into 6163e94

armi/bookkeeping/db/tests/test_comparedb3.py

@@ -94,7 +94,7 @@ def test_diffResultsBasic(self):
         self.assertEqual(dr.nDiffs(), 10)
 
     def test_compareDatabaseDuplicate(self):
-        """end-to-end test of compareDatabases() on a photocopy database."""
+        """End-to-end test of compareDatabases() on a photocopy database."""
         # build two super-simple H5 files for testing
         o, r = test_reactors.loadTestReactor(
             TEST_ROOT, customSettings={"reloadDBName": "reloadingDB.h5"}

armi/cases/suite.py

@@ -203,9 +203,10 @@ def run(self):
         """
         Run each case, one after the other.
 
-        .. warning:: Suite running may not work yet if the cases have interdependencies.
-                    We typically run on a HPC but are still working on a platform
-                    independent way of handling HPCs.
+        Warning
+        -------
+        Suite running may not work yet if the cases have interdependencies. We typically run on a
+        HPC but are still working on a platform independent way of handling HPCs.
         """
         for ci, case in enumerate(self):
             runLog.important(f"Running case {ci+1}/{len(self)}: {case}")
@@ -313,6 +314,7 @@ def writeTable(tableResults):
             userFile, refFile, caseIssues = tableResults[testName]
             data.append((testName, userFile, refFile, caseIssues))
             totalDiffs += caseIssues
+
         print(tabulate.tabulate(data, header, tablefmt=fmt))
         print(
             tabulate.tabulate(

armi/cases/tests/test_cases.py

@@ -20,16 +20,20 @@
 import platform
 import unittest
 
+import h5py
+
 from armi import cases
 from armi import context
 from armi import getApp
 from armi import interfaces
 from armi import plugins
 from armi import runLog
 from armi import settings
+from armi.bookkeeping.db.databaseInterface import DatabaseInterface
 from armi.physics.fuelCycle.settings import CONF_SHUFFLE_LOGIC
 from armi.reactor import blueprints
 from armi.reactor import systemLayoutInput
+from armi.reactor.tests import test_reactors
 from armi.tests import ARMI_RUN_PATH
 from armi.tests import mockRunLogs
 from armi.tests import TEST_ROOT
@@ -307,16 +311,10 @@ def test_checkInputs(self):
         self.c2.checkInputs()
 
     def test_dependenciesWithObscurePaths(self):
-        """
-        Test directory dependence.
-
-        .. tip:: This should be updated to use the Python pathlib
-            so the tests can work in both Linux and Windows identically.
-        """
+        """Test directory dependence for strangely-written file paths (escape characters)."""
         checks = [
             ("c1.yaml", "c2.yaml", "c1.h5", True),
             (r"\\case\1\c1.yaml", r"\\case\2\c2.yaml", "c1.h5", False),
-            # below doesn't work due to some windows path obscurities
             (r"\\case\1\c1.yaml", r"\\case\2\c2.yaml", r"..\1\c1.h5", False),
         ]
         if platform.system() == "Windows":
@@ -335,7 +333,7 @@ def test_dependenciesWithObscurePaths(self):
                         r"c2.yaml",
                         r".\c1.h5",
                         True,
-                    ),  # py bug in 3.6.4 and 3.7.1 fails here
+                    ),
                     (
                         r"\\cas\es\1\c1.yaml",
                         r"\\cas\es\2\c2.yaml",
@@ -436,6 +434,63 @@ def test_buildCommand(self):
         self.assertEqual(cmd, 'python -u  -m armi run "c1.yaml"')
 
 
+class TestCaseSuiteComparison(unittest.TestCase):
+    """CaseSuite.compare() tests."""
+
+    def setUp(self):
+        self.td = directoryChangers.TemporaryDirectoryChanger()
+        self.td.__enter__()
+
+    def tearDown(self):
+        self.td.__exit__(None, None, None)
+
+    def test_compareNoDiffs(self):
+        """As a baseline, this test should always reveal zero diffs."""
+        # Build the cases
+        suite = cases.CaseSuite(settings.Settings())
+
+        geom = systemLayoutInput.SystemLayoutInput()
+        geom.readGeomFromStream(io.StringIO(GEOM_INPUT))
+        bp = blueprints.Blueprints.load(BLUEPRINT_INPUT)
+
+        c1 = cases.Case(cs=settings.Settings(), geom=geom, bp=bp)
+        c1.cs.path = "c1.yaml"
+        suite.add(c1)
+
+        c2 = cases.Case(cs=settings.Settings(), geom=geom, bp=bp)
+        c2.cs.path = "c2.yaml"
+        suite.add(c2)
+
+        # build two super-simple H5 files for testing
+        o, r = test_reactors.loadTestReactor(
+            TEST_ROOT, customSettings={"reloadDBName": "reloadingDB.h5"}
+        )
+
+        # create two DBs, identical but for file names
+        tmpDir = os.getcwd()
+        dbs = []
+        for i in range(1, 3):
+            # create the tests DB
+            dbi = DatabaseInterface(r, o.cs)
+            dbi.initDB(fName=f"{tmpDir}/c{i}.h5")
+            db = dbi.database
+
+            # validate the file exists, and force it to be readable again
+            b = h5py.File(db._fullPath, "r")
+            self.assertEqual(list(b.keys()), ["inputs"])
+            self.assertEqual(
+                sorted(b["inputs"].keys()), ["blueprints", "geomFile", "settings"]
+            )
+            b.close()
+
+            # append to lists
+            dbs.append(db)
+
+        # do a comparison that should have no diffs
+        diff = c1.compare(c2)
+        self.assertEqual(diff, 0)
+
+
 class TestExtraInputWriting(unittest.TestCase):
     """Make sure extra inputs from interfaces are written."""
 

armi/reactor/assemblies.py

@@ -126,8 +126,7 @@ def renameBlocksAccordingToAssemblyNum(self):
 
         Notes
         -----
-        You must run armi.reactor.reactors.Reactor.regenAssemblyLists after calling
-        this.
+        You must run armi.reactor.reactors.Reactor.regenAssemblyLists after calling this.
         """
         assemNum = self.getNum()
         for bi, b in enumerate(self):
@@ -303,7 +302,9 @@ def getPinPlenumVolumeInCubicMeters(self):
         -----
         If there is no plenum blocks in the assembly, a plenum volume of 0.0 is returned
 
-        .. warning:: This is a bit design-specific for pinned assemblies
+        Warning
+        -------
+        This is a bit design-specific for pinned assemblies
         """
         plenumBlocks = self.getBlocks(Flags.PLENUM)
 
@@ -342,8 +343,9 @@ def doubleResolution(self):
         -----
         Used for mesh sensitivity studies.
 
-        .. warning:: This is likely destined for a geometry converter rather than
-            this instance method.
+        Warning
+        -------
+        This is likely destined for a geometry converter rather than this instance method.
         """
         newBlockStack = []
         topIndex = -1
@@ -412,6 +414,7 @@ def adjustResolution(self, refA):
                 newBlocks -= (
                     1  # subtract one because we eliminated the original b completely.
                 )
+
         self.removeAll()
         self.spatialGrid = grids.axialUnitGrid(len(newBlockStack))
         for b in newBlockStack:
@@ -439,7 +442,6 @@ def getAxialMesh(self, centers=False, zeroAtFuel=False):
 
         armi.reactor.reactors.Reactor.findAllAxialMeshPoints : gets a global list of all
         of these, plus finer res.
-
         """
         bottom = 0.0
         meshVals = []
@@ -509,7 +511,6 @@ def getTotalHeight(self, typeSpec=None):
         -------
         height : float
             the height in cm
-
         """
         h = 0.0
         for b in self:
@@ -561,7 +562,6 @@ def getElevationBoundariesByBlockType(self, blockType=None):
         elevation : list of floats
             Every float in the list is an elevation of a block boundary for the block
             type specified (has duplicates)
-
         """
         elevation, elevationsWithBlockBoundaries = 0.0, []
 

armi/reactor/reactors.py

@@ -2350,41 +2350,6 @@ def getAvgTemp(self, typeSpec, blockList=None, flux2Weight=False):
         else:
             raise RuntimeError("no temperature average for {0}".format(typeSpec))
 
-    def getAllNuclidesIn(self, mats):
-        """
-        Find all nuclides that are present in these materials anywhere in the core.
-
-        Parameters
-        ----------
-        mats : iterable or Material
-            List (or single) of materials to scan the full core for, accumulating a nuclide list
-
-        Returns
-        -------
-        allNucNames : list
-            All nuclide names in this material anywhere in the reactor
-
-        See Also
-        --------
-        getDominantMaterial : finds the most prevalent material in a certain type of blocks
-        Block.adjustDensity : modifies nuclides in a block
-
-        Notes
-        -----
-        If you need to know the nuclides in a fuel pin, you can't just use the sample returned
-        from getDominantMaterial, because it may be a fresh fuel material (U and Zr) even though
-        there are burned materials elsewhere (with U, Zr, Pu, LFP, etc.).
-        """
-        if not isinstance(mats, list):
-            # single material passed in
-            mats = [mats]
-        names = set(m.name for m in mats)
-        allNucNames = set()
-        for c in self.iterComponents():
-            if c.material.name in names:
-                allNucNames.update(c.getNuclides())
-        return list(allNucNames)
-
     def growToFullCore(self, cs):
         """Copies symmetric assemblies to build a full core model out of a 1/3 core model.