Skip to content

Add remediation steps to PredictableMetrics healthcheck (#5083)#5083

Open
shrutipatel31 wants to merge 7 commits intofacebook:mainfrom
shrutipatel31:export-D97189120
Open

Add remediation steps to PredictableMetrics healthcheck (#5083)#5083
shrutipatel31 wants to merge 7 commits intofacebook:mainfrom
shrutipatel31:export-D97189120

Conversation

@shrutipatel31
Copy link
Contributor

@shrutipatel31 shrutipatel31 commented Mar 20, 2026

Summary:

When some metrics are unpredictable, add concrete investigation steps

Differential Revision: D97189120

@meta-codesync
Copy link

meta-codesync bot commented Mar 20, 2026

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D97189120.

@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 20, 2026
@meta-codesync meta-codesync bot changed the title Add remediation steps to PredictableMetrics healthcheck Add remediation steps to PredictableMetrics healthcheck (#5083) Mar 20, 2026
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
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
Summary:
Pull Request resolved: facebook#5083

When some metrics are unpredictable, add concrete investigation steps

Differential Revision: D97189120
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant