Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev/cams solar radiation timeseries #190

Merged
merged 18 commits into from
Aug 7, 2024

Conversation

ecmwf-cobarzan
Copy link
Contributor

Here is the list of modifications:

  • refactor and factor in the CAMS-solar-radiation-timeseries adaptor (the initial purpose of this branch)
  • cache avoidance logic (by adding a random key under normalise_request, as recommended by Mattia)
  • call intersect constraints from the multi adaptor and CAMS-solar-radiation-timeseries adaptor (needed some data range support added to the intersect_constraint logic)

The cache avoidance and constraints intersection are triggered using configuration switches (off by default, so that the preexisting behaviour is preserved).

@ecmwf-cobarzan ecmwf-cobarzan requested a review from EddyCMWF August 6, 2024 23:36
@ecmwf-cobarzan ecmwf-cobarzan force-pushed the dev/cams-solar-radiation-timeseries branch from 4bbe0fa to 023bec1 Compare August 7, 2024 11:04
@EddyCMWF EddyCMWF merged commit 1fba937 into main Aug 7, 2024
9 checks passed
@EddyCMWF EddyCMWF deleted the dev/cams-solar-radiation-timeseries branch August 7, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants