Climo Data Unit Conversion Problem #2033
-
When verifying model forecasts of temperature in point_stat using the NCEP climo file for reference, I am having a problem converting the climo_data units. The model data is in Kelvin so I convert it to Celsius during verification. The climo data is also in Kelvin and so I also try to convert it to Celsius. I am using the built in conversion K_to_C(x). The model data and observation data converts fine but the climo data does the following. If I have a Kelvin temperature of 264.15, after the conversion I get -282.15. It seems to have convert the number twice (264.15 - 273.15 - 273.15). You should still have the NCEP climo file from discussion #1879 |
Beta Was this translation helpful? Give feedback.
Replies: 3 comments 2 replies
-
Hello, And thank you for your question. I was able to set up a test in Grid-Stat using MET 11.0.0 and replicate the issue you found. After utilizing the convert(x) = K_to_C(x) utility independently for the forecast and climatology mean dictionaries, as well as both dictionaries, As mentioned, this is not dependent on if the convert utility is used in the forecast field. @JohnHalleyGotway, can you take a look into the code that's responsible for conversion, specifically when it's used in the |
Beta Was this translation helpful? Give feedback.
-
I wanted to follow up that an issue has been created to address this: https://github.com/dtcenter/METplus/issues/2035 In that bugfix, you'll see that @JohnHalleyGotway was able to find a short-term fix for this. By placing the convert function inside the field array, the appropriate conversion is made. This would look something like:
Understandably, this behavior should also occur when the convert function is placed in the |
Beta Was this translation helpful? Give feedback.
-
Hello, I wanted to reach out again and mention that this issue has been resolved. It should be available in the latest bugfix. Take a look here for the specifics: https://github.com/dtcenter/METplus/issues/2035 I'm going to close this Discussion as solved. If you have any additional issues, don't hesitate to open a new Discussions topic. |
Beta Was this translation helpful? Give feedback.
Hello,
I wanted to reach out again and mention that this issue has been resolved. It should be available in the latest bugfix. Take a look here for the specifics: https://github.com/dtcenter/METplus/issues/2035
I'm going to close this Discussion as solved. If you have any additional issues, don't hesitate to open a new Discussions topic.