openclimatefix / ocf_datapipes

OCF's DataPipe based dataloader for training and inference
MIT License
13 stars 11 forks source link

Don't filter missing PV timestamps #233

Closed dfulu closed 10 months ago

dfulu commented 10 months ago

Pull Request

Description

Remove filtering of timestamps as default loading strategy. In many cases we will want to train with realistic amounts of missing data

codecov[bot] commented 10 months ago

Codecov Report

Merging #233 (23b0f41) into main (2c39650) will decrease coverage by 0.01%. Report is 24 commits behind head on main. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #233      +/-   ##
==========================================
- Coverage   80.11%   80.10%   -0.01%     
==========================================
  Files         127      127              
  Lines        5611     5610       -1     
==========================================
- Hits         4495     4494       -1     
  Misses       1116     1116              
Files Coverage Δ
ocf_datapipes/load/pv/pv.py 97.14% <ø> (-0.03%) :arrow_down:

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