fix: properly assign sota_exp_fb before None comparison (microsoft#1037)

you-n-g · web-flow · commit 5d6a927501e9 · 2025-07-08T20:31:21.000+08:00
diff --git a/rdagent/scenarios/data_science/proposal/exp_gen/proposal.py b/rdagent/scenarios/data_science/proposal/exp_gen/proposal.py
@@ -823,7 +823,7 @@ def gen(
                 ]
             )
 
-        if sota_exp_fb := trace.sota_experiment_fb() is None:
+        if (sota_exp_fb := trace.sota_experiment_fb()) is None:
             sota_exp, fb_to_sota_exp = None, None
         else:
             sota_exp, fb_to_sota_exp = sota_exp_fb

Original file line number	Diff line number	Diff line change
`@@ -823,7 +823,7 @@ def gen(`
`823`	`823`	`]`
`824`	`824`	`)`
`825`	`825`
`826`		`- if sota_exp_fb := trace.sota_experiment_fb() is None:`
	`826`	`+ if (sota_exp_fb := trace.sota_experiment_fb()) is None:`
`827`	`827`	`sota_exp, fb_to_sota_exp = None, None`
`828`	`828`	`else:`
`829`	`829`	`sota_exp, fb_to_sota_exp = sota_exp_fb`