openclimatefix / MetOfficeDataHub

Python wrapper around MetOffice Atmospheric Model Data REST API
MIT License
3 stars 0 forks source link

save to s3 and then rename #42

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

Raise condition, when saving to s3, and downloading at the same time, it creates a problem

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

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

Checklist:

codecov-commenter commented 2 years ago

Codecov Report

Merging #42 (4106aa3) into main (2a65810) will increase coverage by 0.08%. The diff coverage is 100.00%.

:exclamation: Current head 4106aa3 differs from pull request most recent head a62dd9f. Consider uploading reports for the commit a62dd9f to get more accurate results

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   93.31%   93.39%   +0.08%     
==========================================
  Files           6        6              
  Lines         329      333       +4     
==========================================
+ Hits          307      311       +4     
  Misses         22       22              
Impacted Files Coverage Δ
metofficedatahub/multiple_files.py 93.00% <100.00%> (+0.20%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

peterdudfield commented 2 years ago

https://github.com/all-contributors please add @jacobbieker for review

peterdudfield commented 2 years ago

@all-contributors please add @jacobbieker for review

allcontributors[bot] commented 2 years ago

@peterdudfield

@jacobbieker already contributed before to review