openclimatefix / pv-site-api

Site specific API for PV forecasting
5 stars 8 forks source link

Issue/gsp dno total #119

Closed peterdudfield closed 8 months ago

peterdudfield commented 10 months ago

Pull Request

Description

Add sum_by method to sum by 'total', 'gsp' or 'dno'

Fixes #104 and #108

Need to first merge #118

How Has This Been Tested?

CI tests + run locally

Checklist:

codecov[bot] commented 10 months ago

Codecov Report

Merging #119 (6fcb7cc) into main (a55d835) will increase coverage by 2.73%. Report is 7 commits behind head on main. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   92.42%   95.16%   +2.73%     
==========================================
  Files          11       11              
  Lines         568      621      +53     
==========================================
+ Hits          525      591      +66     
+ Misses         43       30      -13     
Files Changed Coverage Δ
pv_site_api/__init__.py 100.00% <100.00%> (ø)
pv_site_api/_db_helpers.py 99.19% <100.00%> (+14.08%) :arrow_up:
pv_site_api/convert.py 100.00% <100.00%> (ø)
pv_site_api/main.py 93.59% <100.00%> (+0.03%) :arrow_up:
pv_site_api/utils.py 100.00% <100.00%> (ø)

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