scipp / esssans

SANS data reduction for the European Spallation Source
https://scipp.github.io/esssans/
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Try to speed up tests by caching loaded files #165

Closed SimonHeybrock closed 2 months ago

SimonHeybrock commented 2 months ago

There is probably a way to tweak the caching in the I(Q) beam-center finder to fix the root cause, but modifying that code was already required way too often. I'd like to avoid that for now. Ideally I would like to use the new ess.reduce.streaming.StreamProcessor, but the control flow is just too convoluted to make this easy.

CI run times were at about 16 minutes (https://github.com/scipp/esssans/actions/workflows/ci.yml?query=is%3Asuccess+branch%3Amain), but it probably got worse with recent changes (rising to around 20).

Now tests passed in 9+7 minutes, same as previously. There must be a better way?