fix cygnss quality flag

[BenjaminRuston]

Description

This corrects the get_normalized_bit function in the CyGNSS ingest

Issue(s) addressed

Resolves #1599

Impact

Correct interpretation of quality bits

Checklist

• I have performed a self-review of my own code
• I have run the unit tests before creating the PR

[BenjaminRuston]

think this fixes it... should also check the ascending / descending bit

[BenjaminRuston]

this will also apply to this, and now is a good time to put get_normalized_bit into a utility (https://github.com/JCSDA-internal/ioda-converters/blob/develop/src/pyiodaconv/def_jedi_utils.py):

ioda-converters/src/hdf5/tropics_2ioda.py

Line 420 in a2f5383

return (value >> bit_index) & 1

[BenjaminRuston]

and in the tropics converter I tested the ascending descending flag....

believe this PR introduces the correct method and is what should be used... should confirm this method with the tropics converter too

[BenjaminRuston]

here's the new figure with quality flag applied:

and it's subtle,,, but the counts, min/max and stdv all show improvements in the figure above here's without or no QC bit applied:

[BenjaminRuston]

tested this change with tropics converter too. because it's an array can't wrap the passed in value with int(value) but using it as written and with the == 1 at the end produced the correct land_flag behavior and asc/des behavior:
land flag (mislabeled):

and asc/dsc:

[BenjaminRuston]

think this whole PR is a nothing burger.... just pulled the CyGNSS data from r2d2 for first 5 days in May and the high wind speeds are already screened here's the figures for data previously converted and newly converted and they are identical:

[fabiolrdiniz]

Thanks, @BenjaminRuston!