openclimatefix / nowcasting_datamodel

Datamodel for the nowcasting project

6 stars 6 forks source link

use joinedload to speed up gett all forecasts objects #51

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago

Pull Request

Description

use 'joinedload' to pre load children. This means only one sql command is made. Fixes #48

How Has This Been Tested?

unittests
added a test
[ ] 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