openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 6 forks source link

add extra field to pydantic object for normalized power #49

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

Tidy up normalized power, so the original power is also there

How Has This Been Tested?

unittests

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 #49 (d67f74e) into main (01eeacb) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   98.37%   98.42%   +0.04%     
==========================================
  Files          16       16              
  Lines         617      636      +19     
==========================================
+ Hits          607      626      +19     
  Misses         10       10              
Impacted Files Coverage Δ
nowcasting_datamodel/migrations/app.py 96.66% <ø> (-0.21%) :arrow_down:
nowcasting_datamodel/__init__.py 100.00% <100.00%> (ø)
nowcasting_datamodel/fake.py 100.00% <100.00%> (ø)
nowcasting_datamodel/models/models.py 100.00% <100.00%> (ø)
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