openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

update get_gsp_yield_sum #209

Closed rachel-labri-tipton closed 1 year ago

rachel-labri-tipton commented 1 year ago

Pull Request

Description

Updates get_gsp_yield_sum to return numeric values for solar_generation_kw in GSPYieldSQL.

I also ran black formatter on the code, which explains the updated spacing.

Fixes #208

How Has This Been Tested?

I ran the Docker tests, and all tests in test_read_gsp.py passed.

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #209 (896f2d0) into main (017454b) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   94.98%   94.99%           
=======================================
  Files          27       27           
  Lines        1676     1677    +1     
=======================================
+ Hits         1592     1593    +1     
  Misses         84       84           
Impacted Files Coverage Δ
nowcasting_datamodel/read/read_gsp.py 100.00% <100.00%> (ø)

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