pytroll / satpy / 37504507905
96%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 481
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 96.462% (+0.003%) from 96.459%
37504507905

push

github

web-flow 
Add raw "counts" calibration type to "metimage_l1b_nc" reader (#3373)

* add raw counts calibration type

* whoops.  messed up indenting, and should catch KeyError

* Address review feedback on METimage counts calibration

- counts units: "count" -> "1" per review consensus (all 20 channels)
- restore _FillValue at masked pixels instead of NaN->astype (undefined
  behavior that yielded 0), and round before the integer cast to guard
  against float truncation
- clarify the capital-B solar irradiance fallback: operational products
  use lowercase, pre-launch test granules used capital B
- add unit tests: counts round-trip of stored integers, fill restoration,
  unscaled-variable counts, capital-B fallback

Verified counts bit-identical to on-disk digital numbers on a real
SGA1-VII-1B-RAD operational granule (solar and thermal channels).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EorkxydjrSncteX8QtR477

* Keep _FillValue and valid range on METimage counts, parametrize tests

- Counts calibration moved into _calibrate_counts; the restored fill value is
  now also set as the "_FillValue" attribute (cast to the stored dtype).
- Fall back to missing_value when a variable has no _FillValue, so pixels
  xarray masked via missing_value are restored instead of NaN reaching astype.
- Keep valid_min/valid_max for the counts calibration only: they describe the
  packed on-disk integers, so they are correct for counts and still removed
  for radiance/reflectance/brightness temperature.
- Combine the counts tests with pytest.mark.parametrize and cover the
  missing_value path and the valid range handling.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Move counts test variables into a helper to keep _create_l1b_file small

CodeScene flagged _create_l1b_file as a Large Method after the counts
variables were added inline.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Drop missing_v... (continued)

57476 of 59584 relevant lines covered (96.46%)

3.84 hits per line

Coverage Regressions

Lines Coverage ∆ File
2
96.86
0.08% readers/core/metimage_nc.py
2
96.97
0.89% readers/metimage_l1b_nc.py
Jobs
ID Job ID Ran Files Coverage
1 run- - 37504507905.1 481
96.46
GitHub Action Run
Source Files on build 37504507905