Skip to content

Commit 96cf12b

Browse files
Lena Kashtelyanmeta-codesync[bot]
authored andcommitted
Reorder TL eligibility message: action before caution (#5146)
Summary: Pull Request resolved: #5146 Apply self-review suggestion from D97489418: move the actionable instructions (UI button / `Client.add_transferable_experiment`) before the caution text so users see what to do first, then the warning. Cleaned-up version of D97553476 — excludes accidental merge changes to generation_node.py and test_generation_node.py. Reviewed By: shrutipatel31 Differential Revision: D99385259 fbshipit-source-id: 532fc945e00991b4d40ef6c3c89518fd3142fe2f
1 parent 53be145 commit 96cf12b

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

ax/analysis/healthcheck/transfer_learning_analysis.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -170,14 +170,14 @@ def compute(
170170
f"**{n} eligible source experiment(s)** "
171171
"for transfer learning:\n\n"
172172
f"{exp_lines}\n\n"
173-
"Caution: Only use source experiments that are closely related "
174-
"to your current experiment. "
175-
"Using data from unrelated experiments can lead to negative "
176-
"transfer, which may hurt "
177-
"optimization performance. Review the overlapping parameters "
178-
"before enabling transfer learning. If using the UI, 'Learn from "
173+
"If using the UI, 'Learn from "
179174
"Related Experiments' button is likely available for use; "
180-
"otherwise, use `Client.add_transferable_experiment`."
175+
"otherwise, use `Client.add_transferable_experiment`.\n\n"
176+
"Caution: Only use source experiments that are closely related "
177+
"to your current experiment. Using data from unrelated experiments "
178+
"can lead to negative transfer, which may hurt optimization "
179+
"performance. Review the overlapping parameters before enabling "
180+
"transfer learning."
181181
)
182182

183183
return TransferLearningAnalysisCard(

0 commit comments

Comments
 (0)