From d9be5e6a561ee1937b500ea493891199039b64e8 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Thu, 13 Aug 2026 03:30:18 +0000 Subject: [PATCH] fix(refuters): correct num_simulations in DataSubsetRefuter log message The log line in refute_data_subset() read: 'Refutation over {subset_fraction} simulated datasets of size {N} each' where subset_fraction (e.g. 0.8) was inadvertently passed as the first format argument instead of num_simulations (e.g. 100). Compare the corresponding message in bootstrap_refuter.py which correctly formats num_simulations first. Fix: swap the first positional argument from subset_fraction to num_simulations so the message reads correctly, e.g.: 'Refutation over 100 simulated datasets of size 800.0 each' Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: github-actions[bot] --- dowhy/causal_refuters/data_subset_refuter.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dowhy/causal_refuters/data_subset_refuter.py b/dowhy/causal_refuters/data_subset_refuter.py index 3627423ab1..22dbbfc8b9 100755 --- a/dowhy/causal_refuters/data_subset_refuter.py +++ b/dowhy/causal_refuters/data_subset_refuter.py @@ -120,7 +120,7 @@ def refute_data_subset( logger.info( "Refutation over {} simulated datasets of size {} each".format( - subset_fraction, subset_fraction * len(data.index) + num_simulations, subset_fraction * len(data.index) ) )