From a0625fb2c8b0e08a7f5c6845b695ebfb4daa0947 Mon Sep 17 00:00:00 2001 From: Jammy2211 Date: Fri, 15 May 2026 09:06:03 +0100 Subject: [PATCH] fix: add quick_update kwarg to VisualizerExample.visualize_combined MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Mirror the quick_update plumbing added by commit a1e360567 ("Fix AnalysisFactor.visualize_combined dispatch in FactorGraph"). The base Visualizer.visualize_combined and AnalysisFactor.visualize_combined both accept and forward quick_update=quick_update, but VisualizerExample was missed at the time. Result: any FactorGraphModel fit using the example Analysis class raised TypeError on the first quick-update iteration — broke autofit_workspace_test/scripts/graphical/{simultaneous,hierarchical}.py and autofit_workspace/scripts/{features/graphical_models,cookbooks/multiple_datasets}.py. Pure additive — kwarg defaults to False, body unchanged. Refs #1262 Co-Authored-By: Claude Opus 4.7 (1M context) --- autofit/example/visualize.py | 1 + 1 file changed, 1 insertion(+) diff --git a/autofit/example/visualize.py b/autofit/example/visualize.py index 01723f221..722d4c670 100644 --- a/autofit/example/visualize.py +++ b/autofit/example/visualize.py @@ -179,6 +179,7 @@ def visualize_combined( paths: af.DirectoryPaths, instance: af.ModelInstance, during_analysis: bool, + quick_update: bool = False, ): """ Multiple instances of the `Analysis` class can be summed together, meaning that the model is fitted to all