openclimatefix / PVNet

PVnet main repo
MIT License
17 stars 3 forks source link

fix typo #66

Closed dfulu closed 1 year ago

dfulu commented 1 year ago

Pull Request

Description

Simple typo causing error at end of training. Closes #64

How Has This Been Tested?

A full training run is needed to properly test this, and the change is very minor, so no testing has been done. Will revise if needed.

Checklist:

codecov[bot] commented 1 year ago

Codecov Report

Merging #66 (4db7f0f) into main (e420789) will not change coverage. Report is 1 commits behind head on main. The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   68.15%   68.15%           
=======================================
  Files          22       22           
  Lines        1617     1617           
=======================================
  Hits         1102     1102           
  Misses        515      515           
Files Changed Coverage Δ
pvnet/training.py 0.00% <ø> (ø)
pvnet/utils.py 24.00% <0.00%> (ø)
pvnet/__init__.py 100.00% <100.00%> (ø)

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