openclimatefix / nowcasting-metrics

Repo to automatically run metrics on the nowcasting forecast

0 stars 1 forks source link

make abs goes around total ramp rate value #42

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Fix ramp rate, make sure absolute function goes round all of ramp_rate Fixes #

How Has This Been Tested?

CI tests
tested locally using sql
[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings