openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project
6 stars 7 forks source link

Add new indexes #241

Closed peterdudfield closed 9 months ago

peterdudfield commented 9 months ago

Pull Request

Description

And index on

All are DESC indexes, so latest time loads first

Fixes #237

How Has This Been Tested?

Put it on dev, seems to speed up main forecast/all query

Checklist:

codecov[bot] commented 9 months ago

Codecov Report

Merging #241 (f18488c) into main (f2fa744) will increase coverage by 0.00%. Report is 2 commits behind head on main. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #241   +/-   ##
=======================================
  Coverage   95.24%   95.24%           
=======================================
  Files          27       27           
  Lines        1723     1726    +3     
=======================================
+ Hits         1641     1644    +3     
  Misses         82       82           
Files Changed Coverage Δ
nowcasting_datamodel/models/forecast.py 85.64% <100.00%> (+0.14%) :arrow_up:
nowcasting_datamodel/models/gsp.py 94.44% <100.00%> (+0.07%) :arrow_up:

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