Add remediation steps to PredictableMetrics healthcheck (#5083)#5083
Open
shrutipatel31 wants to merge 7 commits intofacebook:mainfrom
Open
Add remediation steps to PredictableMetrics healthcheck (#5083)#5083shrutipatel31 wants to merge 7 commits intofacebook:mainfrom
shrutipatel31 wants to merge 7 commits intofacebook:mainfrom
Conversation
|
@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D97189120. |
shrutipatel31
added a commit
to shrutipatel31/Ax
that referenced
this pull request
Mar 20, 2026
Summary: When some metrics are unpredictable, add concrete investigation steps Differential Revision: D97189120
c9b84b6 to
9d4e43b
Compare
Summary: Pull Request resolved: facebook#5080 Two message improvements in EarlyStoppingAnalysis Healthcheck : 1. ESS enabled but strategy missing for MOO/constrained: Clarify that this happens when loading a historical experiment without passing the original strategy to EarlyStoppingAnalysis. 2. No MapMetrics case: Prefix with 'Early stopping strategy is enabled, but...' and add action steps to fix the metric configuration. Differential Revision: D97173502
Summary: Pull Request resolved: facebook#5079 Add specific remediation steps to the FAIL case (stale/no trials), guiding users to check generation strategy configuration, parameter constraints, pending trials, and errors. Differential Revision: D97184431
Summary: Pull Request resolved: facebook#5078 Two message improvements: - Partial improvement: Highlight which metrics improved instead of listing which did not. - No improvement: Add actionable next steps. Differential Revision: D97184438
Summary: Add 5-step remediation guide covering metric implementation review, trial configuration, external dependencies, retrying failed fetches, and inspecting tracebacks. Differential Revision: D97184432
Summary: Pull Request resolved: facebook#5081 Add remediation steps for when no effects are detected. Differential Revision: D97184434
Summary: Pull Request resolved: facebook#5082 Add guidance on what to do with regressing arms when warning the users in this healthcheck. Differential Revision: D97184436
shrutipatel31
added a commit
to shrutipatel31/Ax
that referenced
this pull request
Mar 20, 2026
Summary: When some metrics are unpredictable, add concrete investigation steps Differential Revision: D97189120
9d4e43b to
79f6867
Compare
Summary: Pull Request resolved: facebook#5083 When some metrics are unpredictable, add concrete investigation steps Differential Revision: D97189120
79f6867 to
af7bc0b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary:
When some metrics are unpredictable, add concrete investigation steps
Differential Revision: D97189120