armi icon indicating copy to clipboard operation
armi copied to clipboard

Change the properties of Doppler constant params

Open mgjarrett opened this issue 1 year ago • 1 comments

What is the change?

Reactivity coefficient parameters for the Doppler constant were configured with the ParamLocation.AVERAGE location attribute. These parameters are actually ParamLocation.VOLUME_INTEGRATED, so they are being changed.

Why is the change being made?

The Doppler constant for a given block is proportional to the size of the block. If two blocks were to be combined into one, the combined block Doppler constant would be the sum of the two smaller blocks:

b.p.rxFuelDopplerConstant = b1.p.rxFuelDopplerConstant + b2.p.rxFuelDopplerConstant

This is achieved with ParamLocation.VOLUME_INTEGRATED, not PararmLocation.AVERAGE.

The Doppler constant is directly related to the Doppler coefficient in units of pcm/K; the coefficients were already labeled as ParamLocation.VOLUME_INTEGRATED.


Checklist

  • [ ] The release notes have been updated if necessary.
  • [x] The documentation is still up-to-date in the doc folder.
  • [x] The dependencies are still up-to-date in pyproject.toml.

mgjarrett avatar Mar 07 '24 01:03 mgjarrett

What's the status of this PR?

Nominally, it looks like you did what you wanted. Are you waiting to add a unit test or something?

john-science avatar Mar 26 '24 16:03 john-science

@mgjarrett Can you put a short release note under the API Changes section? Thanks!

john-science avatar Jun 07 '24 20:06 john-science

~It's odd this change doesn't require any unit tests changes, right?~

Okay, we have unit tests for things like "make sure VOLUME_INTEGRATED works", but not testing it for each Parameter. I guess that's the best idea I can come up with right now.

john-science avatar Jun 07 '24 20:06 john-science