terrapower / armi

An open-source nuclear reactor analysis automation framework that helps design teams increase efficiency and quality
https://terrapower.github.io/armi/
Apache License 2.0
214 stars 82 forks source link

Fix a few notes within User Manual #1699

Closed albeanth closed 1 month ago

albeanth commented 1 month ago

What is the change?

While reading parts of the UM, I've found a couple notes that are out-of-date.

  1. There is no T/H module within ARMI so temperatures are not updated in ARMI with tight coupling.
  2. Block heights don't have to be input at hot temperatures.

Why is the change being made?

Having correct docs is important.


Checklist

john-science commented 1 month ago

@albeanth We can merge your PR, as-is, but perhaps we could also add this one-line change to the docs in this PR?

An error was noted in section 2.1.4.2 if the ARMI UM. The first line of the code block is

b = o.r.getFirstBlock(Flags.FUEL)

but should be:

b = o.r.core.getFirstBlock(Flags.FUEL)

At your discretion.

albeanth commented 1 month ago

At your discretion.

Done!