SensitivityAnalysisPlot Aesthetic Improvements (#5015)#5015
Closed
eonofrey wants to merge 1 commit intofacebook:mainfrom
Closed
SensitivityAnalysisPlot Aesthetic Improvements (#5015)#5015eonofrey wants to merge 1 commit intofacebook:mainfrom
eonofrey wants to merge 1 commit intofacebook:mainfrom
Conversation
meta-cla
bot
added
the
CLA Signed
meta-codesync
bot
changed the title
eonofrey
added a commit
to eonofrey/Ax
that referenced
this pull request
Mar 12, 2026
Summary: Pull Request resolved: facebook#5015 This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Differential Revision: D96357309
eonofrey
force-pushed
the
export-D96357309
branch
from
2d670c3 to
86aa738
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5015 +/- ##
==========================================
- Coverage 96.75% 96.74% -0.02%
==========================================
Files 606 606
Lines 66152 66210 +58
==========================================
+ Hits 64007 64055 +48
- Misses 2145 2155 +10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
eonofrey
force-pushed
the
export-D96357309
branch
from
86aa738 to
932ef02
Compare
eonofrey
added a commit
to eonofrey/Ax
that referenced
this pull request
Mar 16, 2026
Summary: This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Differential Revision: D96357309
eonofrey
force-pushed
the
export-D96357309
branch
from
932ef02 to
33e9a11
Compare
eonofrey
added a commit
to eonofrey/Ax
that referenced
this pull request
Mar 17, 2026
Summary: This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Differential Revision: D96357309
eonofrey
added a commit
to eonofrey/Ax
that referenced
this pull request
Mar 17, 2026
Summary: This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Reviewed By: mpolson64 Differential Revision: D96357309
eonofrey
force-pushed
the
export-D96357309
branch
from
33e9a11 to
fb8e671
Compare
eonofrey
added a commit
to eonofrey/Ax
that referenced
this pull request
Mar 17, 2026
Summary: This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Reviewed By: mpolson64 Differential Revision: D96357309
eonofrey
force-pushed
the
export-D96357309
branch
from
fb8e671 to
c8d98d5
Compare
Summary: Pull Request resolved: facebook#5015 This diff improves the look of SensitivityAnalysisPlot by: 1. Filtering out `TRIAL_PARAM` (a synthetic paramter from multi-task modeling) which was confusing and not very interpretable. 2. Wrapping long paramter names instead of truncating them with "..." Reviewed By: mpolson64 Differential Revision: D96357309
eonofrey
force-pushed
the
export-D96357309
branch
from
c8d98d5 to
c93249c
Compare
|
This pull request has been merged in f132c71. |
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
2 participants
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:
This diff improves the look of SensitivityAnalysisPlot by:
TRIAL_PARAM(a synthetic paramter from multi-task modeling) which was confusing and not very interpretable.Reviewed By: mpolson64
Differential Revision: D96357309