openclimatefix / power_perceiver

Machine learning experiments using the Perceiver IO model to forecast the electricity system (starting with solar)
MIT License
7 stars 1 forks source link

Change to using OCF Datapipe's BatchKey #198

Closed jacobbieker closed 2 years ago

jacobbieker commented 2 years ago

Pull Request

Description

Turns out Enums can't just be used if they are the same name, this makes them identical by using ocf_datapipe's BatchKey's instead

Fixes #

How Has This Been Tested?

Existing unit tests

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 #198 (e75c2cf) into main (a1cb33e) will decrease coverage by 1.34%. The diff coverage is 84.21%.

@@            Coverage Diff             @@
##             main     #198      +/-   ##
==========================================
- Coverage   39.07%   37.73%   -1.35%     
==========================================
  Files          47       47              
  Lines        2641     2584      -57     
==========================================
- Hits         1032      975      -57     
  Misses       1609     1609              
Impacted Files Coverage Δ
power_perceiver/consts.py 100.00% <ø> (ø)
...tch_processor/delete_forecast_satellite_imagery.py 0.00% <0.00%> (ø)
power_perceiver/np_batch_processor/save_t0_time.py 0.00% <0.00%> (ø)
power_perceiver/production/model.py 33.33% <33.33%> (ø)
...erceiver/load_prepared_batches/data_sources/gsp.py 31.42% <100.00%> (+2.01%) :arrow_up:
...pared_batches/data_sources/prepared_data_source.py 56.86% <100.00%> (ø)
...perceiver/load_prepared_batches/data_sources/pv.py 27.27% <100.00%> (+1.69%) :arrow_up:
...er/load_prepared_batches/data_sources/satellite.py 43.33% <100.00%> (+1.95%) :arrow_up:
...erceiver/load_prepared_batches/data_sources/sun.py 44.44% <100.00%> (+2.13%) :arrow_up:
...erceiver/load_prepared_batches/prepared_dataset.py 34.66% <100.00%> (ø)
... and 17 more

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