pytroll / pytroll-pps-runner

Pytroll runner for PPS
GNU General Public License v3.0
1 stars 8 forks source link

Bugfix xml timestat from ascii #49

Closed adybbroe closed 2 years ago

adybbroe commented 2 years ago

This PR solves an issue with PPS time control files and orbit numbering. PPSv2018 does not pick up the orbit number from the input posttroll message neither the MODIS EOS-HDF files, Therefore the PPS (level-1c and intermediate and final product) files have 00000 as orbit number in the file names. This result in no time-control found and the runner crashes.

codecov[bot] commented 2 years ago

Codecov Report

Merging #49 (25f9b82) into main (debcea1) will increase coverage by 1.47%. The diff coverage is 84.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   56.90%   58.37%   +1.47%     
==========================================
  Files          14       14              
  Lines        1884     1953      +69     
  Branches      243      248       +5     
==========================================
+ Hits         1072     1140      +68     
- Misses        790      791       +1     
  Partials       22       22              
Flag Coverage Δ
unittests 58.37% <84.44%> (+1.47%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
nwcsafpps_runner/utils.py 33.63% <61.11%> (+2.57%) :arrow_up:
nwcsafpps_runner/tests/test_utils.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update debcea1...25f9b82. Read the comment docs.