Ax icon indicating copy to clipboard operation
Ax copied to clipboard

Deprecate Trial.runner

Open eonofrey opened this issue 2 months ago • 2 comments

Summary: Upstream functionality from MultiTypeExperiment._runner_by_trial_type

Full details in: T222906773 and doc, but a summary is:

{F1982739301}

Differential Revision: D83001393

eonofrey avatar Oct 20 '25 23:10 eonofrey

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

meta-codesync[bot] avatar Oct 20 '25 23:10 meta-codesync[bot]

Codecov Report

:x: Patch coverage is 96.05263% with 3 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 96.52%. Comparing base (6ada350) to head (ce8aef8).

Files with missing lines Patch % Lines
ax/core/base_trial.py 80.00% 3 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4460      +/-   ##
==========================================
- Coverage   96.52%   96.52%   -0.01%     
==========================================
  Files         563      563              
  Lines       58836    58834       -2     
==========================================
- Hits        56792    56788       -4     
- Misses       2044     2046       +2     

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Oct 21 '25 00:10 codecov-commenter

This pull request has been merged in facebook/Ax@1f758520e0195dded019ab982819987dba5f6e91.

meta-codesync[bot] avatar Dec 15 '25 19:12 meta-codesync[bot]