search

facebook / Ax

Adaptive Experimentation Platform
https://ax.dev
MIT License
2.38k stars 312 forks source link

Remove Models.ST_MTGP_LEGACY & _NEHVI #3031

Closed saitcakmak closed 2 weeks ago

saitcakmak commented 2 weeks ago

Summary: Cleaning up legacy model entries and updating them to point to the MBM variant of Models.ST_MTGP to retain backwards compatibility.

Also cleaned up a few references of Models.FULLYBAYESIAN... while at it. These all point to corresponding MBM models already.

Differential Revision: D65568608

facebook-github-bot commented 2 weeks ago

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

codecov-commenter commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.58%. Comparing base (647bdc5) to head (a3de922).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3031 +/- ## ========================================== - Coverage 95.60% 95.58% -0.02% ========================================== Files 483 483 Lines 49112 49095 -17 ========================================== - Hits 46954 46928 -26 - Misses 2158 2167 +9 ```

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

facebook-github-bot commented 2 weeks ago

This pull request has been merged in facebook/Ax@22bd1b2740bb19ec6f8e6e85509baf4f5ca30a25.