facebook / Ax

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

Use experiment._properties["total_concurrent_arms"] for determining n #2792

Closed danielcohenlive closed 2 days ago

danielcohenlive commented 3 days ago

Summary: In GS.gen_for_multiple_trials_with_multiple_models and GS.gen_with_multiple_nodes

Reviewed By: lena-kashtelyan

Differential Revision: D63422137

facebook-github-bot commented 3 days ago

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

codecov-commenter commented 3 days ago

Codecov Report

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

Project coverage is 95.68%. Comparing base (a144287) to head (d82f589).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2792 +/- ## ======================================= Coverage 95.68% 95.68% ======================================= Files 488 488 Lines 47985 48013 +28 ======================================= + Hits 45915 45943 +28 Misses 2070 2070 ```

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

facebook-github-bot commented 2 days ago

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

facebook-github-bot commented 2 days ago

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

facebook-github-bot commented 2 days ago

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