Sync -DRAFT

[peterdudfield]

Pull Request

Description

• Changed all routes to sync not async.
• use grpc_request
• fix: to allow forecast/all with one timestamp to be called, and not go to cache
• Reduced cache to 2 hours (but could increase forecast/all default to 24 hours again)

TODO

• tidy up some formatting
• general tidying
• Add trace_id back in

Memory used to be

Now its
(deployed after 9.15)

You can also see the difference in API performance

Helps with https://github.com/openclimatefix/client-private/issues/294

How Has This Been Tested?

• CI test
• ran locally, and cpu much lower (check)
• ran on dev, and cpu much lower (check)
• clicked around on UI, and it seems faster

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]

Proof in #255 i think shows that we can keep the fastapi async, but just use grpc-requests instead

[peterdudfield]

Ill clsoe this, as we do want to have fastapi working with async. Solution seems to be to used grpc-requests