Block.getNumPins broken for C5G7 model :: multiple cladding components per lattice site

[drewj-tp]

If you load the C5G7 test reactor and grab an assembly, you get a hilariously large number of pins

>>> r = armi.init(fName="armi/tests/c5g7/c5g7-settings.yaml").r
>>> fuelBlock = r.core[0][0]
>>> fuelBlock.getNumPins()
528

In that model, there are two clad components per lattice site

armi/armi/tests/c5g7/c5g7-blueprints.yaml

Lines 86 to 114 in c6f46e5

	zirconium clad: &clad_Zr
	shape: Circle
	material: Custom
	isotopics: Zr clad
	Tinput: 20.0
	Thot: 20.0
	id: .8360
	od: .9500
	latticeIDs: [U]
	gap 2: &fuel_gap_2
	shape: Circle
	material: Void
	Tinput: 20.0
	Thot: 20.0
	id: zirconium clad.od
	od: aluminum clad.id
	latticeIDs: [U]
	aluminum clad: &clad_Al
	# NEA/NSC/DOC(96)2 Documents:
	# "This clad is used to simulate hot conditions at room temperature
	# (decrease the moderation ratio)"
	shape: Circle
	material: Custom
	isotopics: Al clad
	Tinput: 20.0
	Thot: 20.0
	id: .9700
	od: 1.0800
	latticeIDs: [U]

So you double count the sites. There are 17x17 total lattice sites, some of which are empty, so the upper bound on "real" pins is 289

Originally posted by @drewj-tp in #1900 (comment)

[john-science]

A great place for to start would be to write a unit test that can go with the existing C5G7 unit test for this exact thing:

armi/armi/tests/test_lwrInputs.py

Lines 26 to 29 in c6f46e5

	TEST_INPUT_TITLE = "c5g7-settings.yaml"
	class C5G7ReactorTests(unittest.TestCase):

I see you use armi.init() in the above example, that is a no-no for unit tests, so I suggest something like this:

    def test_numFuelPinsC5G7(self):
        _o, r = test_reactors.loadTestReactor(
            os.path.join(TEST_ROOT, "c5g7"),
            inputFileName=TEST_INPUT_TITLE,
        )
        b = r.core.getFirstBlock(Flags.FUEL)
        self.assertEqual(b.getNumPins(), 264)

Or something like that.