Open mpolson64 opened 1 day ago
This pull request was exported from Phabricator. Differential Revision: D65234856
This pull request was exported from Phabricator. Differential Revision: D65234856
Attention: Patch coverage is 94.50172%
with 16 lines
in your changes missing coverage. Please review.
Project coverage is 95.73%. Comparing base (
1efbcbd
) to head (4e95ad2
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
Summary: When this is landed we will be able to use this plot in Ax 1.0 and Ax UI.
Refactor the interaction plot to be in line with our structure for ax.analysis. This includes a massive reduction in overall code (about half) and a full decoupling from ax.plot.
Adds robustness features around generating subplots -- a failed surface subplot will no longer fail the full analysis.
This new version of the plot is slightly more opinionated in that we always plot both the feature importance bar chart AND the top 6 features, always plots top 15 components in the bar chart, never decomposes components, and always has plots share scale. These settings are most useful and help drastically simplify the code, so I think we should keep them for now and only consider adding them back if there is demand.
Differential Revision: D65234856