Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Alex Lindsay <[email protected]>
  • Loading branch information
gsgall and lindsayad committed Aug 6, 2024
1 parent 1bf578c commit fa2a859
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions include/postprocessors/MultiPeriodAverager.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ class MultiPeriodAverager : public GeneralPostprocessor
/// The previous post process value of the post processor we are averaging over several periods
const PostprocessorValue & _pps_value_old;
/// The number of periods that have passed
uint _period_count;
unsigned int _period_count;
/// The counter for how many periods have passed since we last updated
uint _cyclic_period_count;
unsigned int _cyclic_period_count;
/// the number of periods over which we are averaging
uint _num_periods;
unsigned int _num_periods;
};
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class PeriodicTimeIntegratedPostprocessor : public MultipliedTimeIntegratedPostp
/// the period of time over which to integrate
const Real _period;
/// the total number of periods that have occured
uint _period_count;
unsigned int _period_count;
/// the point in the time when the next period begins
Real _next_period_start;
};
6 changes: 3 additions & 3 deletions src/postprocessors/MultiPeriodAverager.C
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ MultiPeriodAverager::validParams()
InputParameters params = GeneralPostprocessor::validParams();
params.addClassDescription(
"Calculate the average value of a post processor over multiple periods");
params.addRangeCheckedParam<uint>("number_of_periods",
"number_of_periods > 0",
"The number of periods over which you are averaging");
params.addRangeCheckedParam<unsigned int>("number_of_periods",
"number_of_periods > 0",
"The number of periods over which you are averaging");
params.addParam<PostprocessorName>(
"value", "The name of the postprocessor you would like to average over multiple periods");
params.addRequiredRangeCheckedParam<Real>(
Expand Down

0 comments on commit fa2a859

Please sign in to comment.