-
Notifications
You must be signed in to change notification settings - Fork 235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor velocity postprocessor output #4342
Refactor velocity postprocessor output #4342
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good otherwise!
for (std::map<types::boundary_id, double>::const_iterator | ||
p = global_max_vel.begin(), a = global_min_vel.begin(), rms = global_rms_vel.begin(); | ||
p != global_max_vel.end() && a != global_min_vel.end() && rms != global_rms_vel.end(); | ||
++p, ++a, ++rms, ++index) | ||
{ | ||
if (this->convert_output_to_years() == true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you know what the meaning of p and a is? Can you replace the variable names please?
Yes, that is the risk when fiddling around with one of the oldest postprocessors :-). Let's see what the tester thinks about this new version. |
This refactors some of our velocity statistics postprocessors to remove code duplication caused by unit conversion between seconds and years. Also assert a special behavior of the boundary_velocity_residual_statistics postprocessor, which looks like it is prepared to work for several boundaries simultaneously, which we did not test so far (currently it is hard-coded to only compute the statistics for the surface boundary).