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.

Sign up for GitHub

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

Jump to bottom

issue #280 adding templates 4.137 to 4.142 and update of code table 4.0 #300

Merged
merged 8 commits into from
Feb 6, 2025
Merged

issue #280 adding templates 4.137 to 4.142 and update of code table 4.0 #300

merged 8 commits into from
Feb 6, 2025

Conversation

amilan17
Copy link
Member

@amilan17 amilan17 commented Jan 20, 2025 •
edited
Loading

amilan17
amilan17 commented Jan 23, 2025
View reviewed changes
Copy link
Member Author

@amilan17 amilan17 left a comment

Choose a reason for hiding this comment

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

@sebvi Octet counts are missing from 4.141

@amilan17 amilan17 linked an issue Jan 23, 2025 that may be closed by this pull request
New templates for reforecast in section 4 #280
Closed
@SibylleK
Update GRIB2_Template_4_137_ProductDefinitionTemplate_en.csv
1370a93 
Some incorrect OctetCount entries corrected
@SibylleK
Update GRIB2_Template_4_138_ProductDefinitionTemplate_en.csv
36eb468 
Delete OctetCount = 12 for the "next innermost step of processing", as it is not a 12 byte  long entry.
@SibylleK
Update GRIB2_Template_4_141_ProductDefinitionTemplate_en.csv
697bea7 
Add OctetCount entries
@SibylleK
Update GRIB2_Template_4_137_ProductDefinitionTemplate_en.csv
02243b1 
Correct typo in month of model ..
@amilan17 amilan17 marked this pull request as draft January 29, 2025 13:33
@amilan17 amilan17 marked this pull request as ready for review February 6, 2025 15:18
@amilan17 amilan17 merged commit d67fc44 into FT2025-1 Feb 6, 2025
1 check failed
@amilan17 amilan17 deleted the 280-new-templates-for-reforecast-in-section-4 branch February 6, 2025 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

New templates for reforecast in section 4
3 participants
@amilan17 @sebvi @SibylleK