openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project

6 stars 6 forks source link

make sure forecast values are ordered by targettime #67

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

make sure that target times are sorted

Fixes #

How Has This Been Tested?

added tests where data is not sorted in the database, this faield before doing the change

[x] Yes

Checklist:

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