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

more unit tests for code in gbits.c, gridtemplates, drstemplates, pdstemplates, plus doxygen improvements and code cleanup #147

Merged
merged 36 commits into from
Oct 25, 2021

Conversation

edwardhartnett
Copy link
Contributor

@edwardhartnett edwardhartnett commented Oct 23, 2021

Part of #8.

More unit tests for the functions in gbits.c and gridtemplates.c. Also documentation improvements.

@edwardhartnett edwardhartnett changed the title more tests more unit tests for code in gbits.c Oct 24, 2021
@edwardhartnett edwardhartnett marked this pull request as ready for review October 24, 2021 13:26
@edwardhartnett
Copy link
Contributor Author

OK, I will stop adding testing on this branch, and start a new one. THis branch takes testing up to ~20% coverage:
image

@edwardhartnett edwardhartnett changed the title more unit tests for code in gbits.c more unit tests for code in gbits.c, gridtemplates, drstemplates, pdstemplates, plus doxygen improvements and code cleanup Oct 25, 2021
@edwardhartnett edwardhartnett merged commit 1f46c45 into develop Oct 25, 2021
*
* @param[in] cgrib Character array to contain the GRIB2 message. Must
* be allocated large enough to store the entire GRIB2 message.
* @param[in] listsec0 Contains information needed for GRIB Indicator
* Section 0. Must be dimensioned >= 2.
* - listsec0[0]=Discipline-GRIB Master Table Number (see Code Table 0.0)
* - listsec0[1]=GRIB Edition Number (currently 2)
* - listsec0(0) Discipline-GRIB Master Table Number ([Code Table 0.0]

Choose a reason for hiding this comment

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

Square bracket

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next PR...

Comment on lines +32 to +41
* - listsec1(0) Id of orginating centre ([Table 0]
* (https://www.nco.ncep.noaa.gov/pmb/docs/on388/table0.html)).
* - listsec1(1) Id of orginating sub-centre ([Table C]
* (https://www.nco.ncep.noaa.gov/pmb/docs/on388/tablec.html)).
* - listsec1(2) GRIB Master Tables Version Number ([Table 1.0]
* (https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table1-0.shtml)).
* - listsec1(3) GRIB Local Tables Version Number ([Table 1.1]
* (https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table1-1.shtml)).
* - listsec1(4) Significance of Reference Time ([Table 1.2]
* (https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table1-2.shtml))

Choose a reason for hiding this comment

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

Square brackets

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next PR...

@edwardhartnett edwardhartnett deleted the ejh_t1 branch October 25, 2021 13:34
Comment on lines +48 to +51
* - listsec1(11) Production status of data ([Table 1.3]
* (https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table1-3.shtml)).
* - listsec1(12) Type of processed data ([Table 1.4]
* (https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table1-4.shtml)).

Choose a reason for hiding this comment

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

Square brackets

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in next PR...

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.

2 participants