openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project

6 stars 6 forks source link

dont add column to partition table #212

Closed peterdudfield closed 1 year ago

peterdudfield commented 1 year ago

Pull Request

Description

Fix migration for adding 'properties' Fixes #

How Has This Been Tested?

ran the mgiration
@rachel-labri-tipton ran analysis dashbaord UI that uses that column
[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
[ ] 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 1 year ago

ONe test is failing and ive put it in here to be sovled - https://github.com/openclimatefix/nowcasting_datamodel/issues/213