Skip to content

Remove Dask from stochastic noise plugin#2363

Open
maxwhitemet wants to merge 5 commits into
metoppv:masterfrom
maxwhitemet:remove_dask_from_stochastic_noise
Open

Remove Dask from stochastic noise plugin#2363
maxwhitemet wants to merge 5 commits into
metoppv:masterfrom
maxwhitemet:remove_dask_from_stochastic_noise

Conversation

@maxwhitemet
Copy link
Copy Markdown
Contributor

Addresses #1131

Description
This PR removes the use of Dask from the stochastic noise plugin, and adds two processing options:

  • supply a cube with a realization dimension to iterate over serially.
  • extract (before using this plugin) each realization into its own cube, run each through the plugin with a scheduler, and merge (after using this plugin) the cubes together.

Testing:

  • Ran tests and they passed OK
  • Added new tests for the new feature(s)

@maxwhitemet maxwhitemet changed the title Remove parallelisation Remove Dask from stochastic noise plugin Apr 20, 2026
Copy link
Copy Markdown
Contributor

@gavinevans gavinevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @maxwhitemet 👍

I've added one comment.

Comment thread improver_tests/calibration/stochastic_noise/test_StochasticNoise.py Outdated
Copy link
Copy Markdown
Contributor Author

@maxwhitemet maxwhitemet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gavinevans. I have implemented the suggested changes

Comment thread improver_tests/calibration/stochastic_noise/test_StochasticNoise.py Outdated
@maxwhitemet maxwhitemet force-pushed the remove_dask_from_stochastic_noise branch from 085c0c5 to f92ead9 Compare April 29, 2026 13:31
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