Skip to content
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

Bug fix for QPFFFG and QPFARI encoding in RRFS #961

Closed
wants to merge 6 commits into from

Conversation

EricJames-NOAA
Copy link
Contributor

prob_num and tot_num_prob were not set in grib2_module.f, resulting in seemingly random values showing up for the new GRIB2 records QPFARI and QPFFFG, as noted by Hui-Ya Chuang:

< 772:4824832913:vt=2024022023:surface:11-12 hour acc fcst:var discipline=1 center=7 local_table=1 parmcat
=1 parm=197:prob >1:prob fcst 72/110

772:4824832913:vt=2024022023:surface:11-12 hour acc fcst:var discipline=1 center=7 local_table=1 parmcat
=1 parm=197:prob >1:prob fcst 96/110
6177c6177
< 773:4824833141:vt=2024022023:surface:0-12 hour acc fcst:var discipline=1 center=7 local_table=1 parmcat=
1 parm=197:prob >1:prob fcst 72/110


773:4824833141:vt=2024022023:surface:0-12 hour acc fcst:var discipline=1 center=7 local_table=1 parmcat=
1 parm=197:prob >1:prob fcst 96/110

This small change fixes the issue, and should lead to bitwise identical results.

The change was tested for the RRFS_NA_3km system on Hera.

@HuiyaChuang-NOAA
Copy link
Collaborator

@EricJames-NOAA @MatthewPyle-NOAA
Wen won't be back for another week or two from China. She gave me directions to merge UPP codes and Fernando will help me.
I will be gone for 3 weeks after this weekend so let me know soon.

Huiya

@EricJames-NOAA
Copy link
Contributor Author

@EricJames-NOAA @MatthewPyle-NOAA Wen won't be back for another week or two from China. She gave me directions to merge UPP codes and Fernando will help me. I will be gone for 3 weeks after this weekend so let me know soon.

Huiya

@HuiyaChuang-NOAA thanks for the update. I am working on one additional bug fix which I intend to combine with this PR. I hope to have the code ready to go tomorrow. Once that is finished, the code should be finalized for RRFS.

@HuiyaChuang-NOAA
Copy link
Collaborator

@EricJames-NOAA @MatthewPyle-NOAA Wen won't be back for another week or two from China. She gave me directions to merge UPP codes and Fernando will help me. I will be gone for 3 weeks after this weekend so let me know soon.
Huiya

@HuiyaChuang-NOAA thanks for the update. I am working on one additional bug fix which I intend to combine with this PR. I hope to have the code ready to go tomorrow. Once that is finished, the code should be finalized for RRFS.

Ok, let me know. I will see what I can do. If I can't get to it, I will email Wen with you copied . She is hoping to be back to work the week of June 10

Copy link
Contributor

@MatthewPyle-NOAA MatthewPyle-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from my perspective.

@EricJames-NOAA
Copy link
Contributor Author

@EricJames-NOAA @MatthewPyle-NOAA Wen won't be back for another week or two from China. She gave me directions to merge UPP codes and Fernando will help me. I will be gone for 3 weeks after this weekend so let me know soon.
Huiya

@HuiyaChuang-NOAA thanks for the update. I am working on one additional bug fix which I intend to combine with this PR. I hope to have the code ready to go tomorrow. Once that is finished, the code should be finalized for RRFS.

Ok, let me know. I will see what I can do. If I can't get to it, I will email Wen with you copied . She is hoping to be back to work the week of June 10

@EricJames-NOAA @MatthewPyle-NOAA Wen won't be back for another week or two from China. She gave me directions to merge UPP codes and Fernando will help me. I will be gone for 3 weeks after this weekend so let me know soon.
Huiya

@HuiyaChuang-NOAA thanks for the update. I am working on one additional bug fix which I intend to combine with this PR. I hope to have the code ready to go tomorrow. Once that is finished, the code should be finalized for RRFS.

Ok, let me know. I will see what I can do. If I can't get to it, I will email Wen with you copied . She is hoping to be back to work the week of June 10

@HuiyaChuang-NOAA @MatthewPyle-NOAA the PR is ready. The only remaining issue is that accumulated fields in the top-of-the-hour 15-min output appear with units of hours instead of minutes, and I can't figure out why that's the case. But the time labels are all correct now.

Sorry it is so late. If you can't get it merged now, I will spend some more time trying to figure out the time unit labeling issue.

@WenMeng-NOAA WenMeng-NOAA linked an issue Jun 9, 2024 that may be closed by this pull request
@WenMeng-NOAA
Copy link
Collaborator

@EricJames-NOAA I assume all of these bug fixes are intended for RRFS v1 implementation, right? If so, please change the merging branch from 'develop' to 'release/rrfs_v1'. Also, another UPP issue is needed for bug fixes except QPFFG and QPFARI encoding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants