FluxFiles Deserve Hashes Too!

opotowsky commented 5 months ago

What is the change?

Adding fluxFiles to one of the crossSectionControl items that is gifted with a SHA.

Why is the change being made?

We want to be able to trace inputs such as these internally.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices. 😈

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
[x] The documentation is still up-to-date in the doc folder.
[x] If any requirements were affected, mention it in the release notes.
[x] The dependencies are still up-to-date in pyproject.toml.

mgjarrett commented 5 months ago

@opotowsky it looks good to me. I don't think we need any changes to the code you added, but I did leave a comment about potentially changing some of the pre-existing variable names.

opotowsky commented 5 months ago

Ok I just reran the case submission internally and our hashes are lookin good! @john-science this can go in whenever you want it to (fine to push it to after the impl PRs are in)

john-science commented 5 months ago

I retroactively approve this PR. :)

terrapower / armi