-
Notifications
You must be signed in to change notification settings - Fork 52
feat: abstracting accumulation #326
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
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
0c78564 to
abd784d
Compare
|
This PR is finished from my point of view and could be merged. Many additional tests have been added and all relevant tests are passing. The API changes slightly and is documented. Another PR will be required to handle other sources and/or compute running averages, as this may lead to changing the argument passed to the sources and needs to be coordinated with the current work on observations. |
I mean description of this PR to get a helpful commit message. |
Hey @MeraX description is now updated! |
Description
This PR revisits the accumulation source to extend it to work with other backends apart from MARS. As part of this work the main outcomes and changes are:
Future work will be done to extend this source to work with other backends. The new design will make this much easier as new backends will be easier to simply plug to the source.
📚 Documentation preview 📚: https://anemoi-datasets--326.org.readthedocs.build/en/326/