Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve robustness in InteractionAnalysis #3407

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mpolson64
Copy link
Contributor

Summary:
A number of features to improve reliablity of the InteractionAnalysis.

  • Hide OAK kernel behind a flag which defaults to False. When false, use the current GenerationNode's adapter
  • If ax_parameter_sens fails log an exception and fallback to the surrogate's feature_importances
  • Do not plot samples on the slice and countour plots if there are more than 50 samples (it gets too cluttered)
  • Changed the orange and blue colors on the importance bar chart to be in the plotly color scheme
  • Make plot not error out on unordered choice params
  • Improved subtitle

Differential Revision: D69993111

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Feb 21, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D69993111

Summary:

As titled. Added comments as well to show whats going on.

Differential Revision: D69987525
Summary:

As titled. These are moderately useful here, but I really want to add them to the contour plot and I want the slice plot to match since theyre often displayed together.

Differential Revision: D69988707
Summary:

As titled.

Differential Revision: D69989849
Summary:

A number of features to improve reliablity of the InteractionAnalysis.
* Hide OAK kernel behind a flag which defaults to False. When false, use the current GenerationNode's adapter
* If ax_parameter_sens fails log an exception and fallback to the surrogate's feature_importances
* Do not plot samples on the slice and countour plots if there are more than 50 samples (it gets too cluttered)
* Changed the orange and blue colors on the importance bar chart to be in the plotly color scheme
* Make plot not error out on unordered choice params
* Improved subtitle

Differential Revision: D69993111
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D69993111

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants