fix: Don't run analysis modules if configuration contents haven't changed

disorderedmaterials / dissolve

Structure refinement software for total scattering data

GNU General Public License v3.0

9 stars 16 forks source link

fix: Don't run analysis modules if configuration contents haven't changed #1935

Closed trisyoungs closed 2 weeks ago

trisyoungs commented 1 month ago

After merging of #1789 and #1795, analysis modules no longer used a Procedure. This means that we are now bypassing the check on a configuration's version to ensure that its contents have changed since the last run of the module.

This PR reintroduces that check in a more general way across all modules.

trisyoungs commented 1 month ago

@rprospero Re-requesting a review on this one since it turns out things didn't quite work as expected following further testing!