Satip
Satip copied to clipboard
Monthly Zarr processing script
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
Codecov Report
Merging #125 (5998286) into main (0ad6a39) will decrease coverage by
0.98%
. The diff coverage is64.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
We are doing it differently now, so this isn't needed