openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project

6 stars 6 forks source link

Add option to run for PV database as well #70

Closed jacobbieker closed 2 years ago

jacobbieker commented 2 years ago

Pull Request

Description

The local infra stack tests fail, see https://github.com/openclimatefix/nowcasting_infrastructure/pull/105 and one of the main issues is that the PV database migrations aren't ran, so all the PV systems tests fail. This fixes that

Fixes #

How Has This Been Tested?

The tests in https://github.com/openclimatefix/nowcasting_infrastructure/pull/105 show that the PV errors do not exist now after using a custom build of datamodel that runs the PV systems

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] 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

peterdudfield commented 2 years ago

Sorry, this can be removed, and simple runs both migrations always