Remove various args from InteractionPlot.__init__

[mpolson64]

Summary: Simplifying InteractionPlot by removing some arguments from its initializer. IMO the typical user just wants to specify a metric (or have the appropriate metric selected for them) and run compute. In some cases an advanced user may want to decide whether or not to plot interactions, but by default we should.

No matter what we should always show the bar chart displaying relative importance of each parameter and the top 6 slices/surfaces.

If this simplifications seem appropriate then I will continue to refactor the InteractionPlot's code to take advantage of this removed flexibility.

Removals:

• top_k: Prefer to show sobol indices for all components on bar chart, slice/sufrace for top 6 always
• data: Let's always use the data on the experiment
• display_components: Always display components
• decompose_components: Never decompose components
• plots_share_range: Always share range
• num_mc_samples: Always use 10k samples. Fwiw this was only added to speed up a unit test, which could be done effectively without this arg by using mocks

The following diffs will restructure the code here to take advantage of the simplified options

Differential Revision: D65148289

[facebook-github-bot]

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

[facebook-github-bot]

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

[facebook-github-bot]

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

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 94.53925% with 16 lines in your changes missing coverage. Please review.

Project coverage is 95.42%. Comparing base (4aa1d65) to head (2704529).

Files with missing lines	Patch %	Lines
ax/analysis/plotly/surface/utils.py	57.14%	9 Missing :warning:
ax/analysis/plotly/surface/contour.py	93.61%	3 Missing :warning:
ax/analysis/plotly/interaction.py	97.59%	2 Missing :warning:
ax/analysis/plotly/surface/slice.py	95.91%	2 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #3096 +/- ## ========================================== - Coverage 95.43% 95.42% -0.01% ========================================== Files 495 503 +8 Lines 49956 50249 +293 ========================================== + Hits 47674 47952 +278 - Misses 2282 2297 +15 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles

[facebook-github-bot]

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

[facebook-github-bot]

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

[facebook-github-bot]

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

[facebook-github-bot]

This pull request has been merged in facebook/Ax@8fe3fed3bdffd526330138ac65a9c25eeb6e4748.