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

Refactor parallelization of bias calculations #227

Open
castelao opened this issue Jul 15, 2024 · 0 comments · May be fixed by #253
Open

Refactor parallelization of bias calculations #227

castelao opened this issue Jul 15, 2024 · 0 comments · May be fixed by #253
Assignees
Labels

Comments

@castelao
Copy link
Member

castelao commented Jul 15, 2024

Why this feature is necessary:

From discussions on #215, @bnb32 suggested to register as a ToDo so we don't forget about it.

A possible solution is:

Different possibilities:

  • Combine all the redundant code on parallelization with futures;
  • Move to xarray and reduce a lot of code while parallelizing with Dask;

I have considered the following alternatives:

For now it is just redundant and explicit so it is prepared for an easy and clean refactoring.

Additional context

None

Charge code

Urgency / Timeframe

Not urgent. It works as it is.

@bnb32 bnb32 self-assigned this Jan 4, 2025
bnb32 added a commit that referenced this issue Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants