Satip icon indicating copy to clipboard operation
Satip copied to clipboard

Monthly Zarr processing script

Open jacobbieker opened this issue 2 years ago • 1 comments

Pull Request

Description

Please delete the italicised instruction text! Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • [ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • [ ] Yes

Checklist:

  • [ ] My code follows OCF's coding style guidelines
  • [ ] I have performed a self-review of my own code
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] I have checked my code and corrected any misspellings

jacobbieker avatar Jul 28 '22 14:07 jacobbieker

Codecov Report

Merging #125 (5998286) into main (0ad6a39) will decrease coverage by 0.98%. The diff coverage is 64.28%.

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
- Coverage   66.69%   65.71%   -0.99%     
==========================================
  Files          10       10              
  Lines        1039     1047       +8     
==========================================
- Hits          693      688       -5     
- Misses        346      359      +13     
Impacted Files Coverage Δ
satip/eumetsat.py 75.62% <61.53%> (+0.49%) :arrow_up:
satip/app.py 90.90% <100.00%> (ø)
satip/download.py 60.97% <0.00%> (-6.71%) :arrow_down:
satip/utils.py 71.14% <0.00%> (-0.29%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Jul 29 '22 09:07 codecov-commenter

We are doing it differently now, so this isn't needed

jacobbieker avatar Feb 14 '23 11:02 jacobbieker