openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

Issue/speed up #47

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

speed up options for api

How Has This Been Tested?

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 #47 (ea2e647) into main (f2756ad) will increase coverage by 0.01%. The diff coverage is 100.00%.

:exclamation: Current head ea2e647 differs from pull request most recent head 2dfebc9. Consider uploading reports for the commit 2dfebc9 to get more accurate results

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   98.40%   98.42%   +0.01%     
==========================================
  Files          16       16              
  Lines         628      635       +7     
==========================================
+ Hits          618      625       +7     
  Misses         10       10              
Impacted Files Coverage Δ
nowcasting_datamodel/read/read.py 99.27% <100.00%> (+0.03%) :arrow_up:

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