-
Notifications
You must be signed in to change notification settings - Fork 63
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
precip_rate now computed using the right unit of temperature #389
Conversation
Looks like the Fortran tests--maybe others--have expected result values based on the old incorrect |
I'm updating those now... |
168f33d
to
b7d6108
Compare
… of temperature
b7d6108
to
4983cb4
Compare
@@ -335,7 +335,7 @@ TEST_F(Bmi_Cpp_Formulation_Test, GetOutputLineForTimestep_1_b) { | |||
formulation.get_response(i++, 3600); | |||
formulation.get_response(i, 3600); | |||
std::string output = formulation.get_output_line_for_timestep(i, ","); | |||
ASSERT_EQ(output, "0.000000,0.000002"); | |||
ASSERT_EQ(output, "0.000000,0.000001"); |
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.
I'll be kind of surprised if the -0
thing only happens on C and not C++ ... but we can keep an eye on it.
Discovered a bug in the script used to generate the example forcing files. The density of water is computed using the following function:
But the AORC temperature variable is in kelvin, and was being passed without converting to celsius.
Fixed this in the script and regenerated the example forcing data files to get correct
precip_rate
(in m/s)Changes
precip_rate
columnChecklist