Skip to content

Commit 0d3535e

Browse files
Lena Kashtelyanmeta-codesync[bot]
authored andcommitted
Make TL eligibility health-check more actionable (facebook#5085)
Summary: Pull Request resolved: facebook#5085 I was very excited to see this warning in the UI: {F1987030629} But it also seemed insufficiently actionable for either users or agents. I think a vague mention of the UI is ok even though this is going into OSS; what do folks think? NOTE: Also a thought: should we be saying "Transfer Learning Eligibilty" or "Meta-Learning Eligibility"? Created from CodeHub with https://fburl.com/edit-in-codehub Reviewed By: mgarrard, eonofrey Differential Revision: D97489418 fbshipit-source-id: b90a0eab1445ad10683baeb917fae81aa9b077d3
1 parent 33e9a5e commit 0d3535e

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

ax/analysis/healthcheck/transfer_learning_analysis.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,9 @@ def compute(
175175
"Using data from unrelated experiments can lead to negative "
176176
"transfer, which may hurt "
177177
"optimization performance. Review the overlapping parameters "
178-
"before enabling transfer learning."
178+
"before enabling transfer learning. If using the UI, 'Learn from "
179+
"Related Experiments' button is likely available for use; "
180+
"otherwise, use `Client.add_transferable_experiment`."
179181
)
180182

181183
return TransferLearningAnalysisCard(

0 commit comments

Comments
 (0)