Refactor data-loading for LCBench surrogate benchmark problem
This diff improves the LCBench data-loading and benchmark problem setup. Key changes include:
- Introducing an alternative constructor option for
LearningCurveBenchmarkTestFunctioninget_lcbench_early_stopping_benchmark_problem - Making data retrieval in
LearningCurveBenchmarkTestFunctionmore flexible with the override-able method_load_lcbench_data - Simplifying the benchmark problem by removing unused options (
observe_noise_sd,noise_std,seed) - Updating naming conventions for LCBench benchmark problems
- Various minor improvements, including type annotations and refactoring of repetitive code
Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.03%. Comparing base (
da222b1) to head (19790da).
Additional details and impacted files
@@ Coverage Diff @@
## main #3634 +/- ##
=======================================
Coverage 96.03% 96.03%
=======================================
Files 564 564
Lines 56305 56300 -5
=======================================
- Hits 54074 54070 -4
+ Misses 2231 2230 -1
: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.
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request was exported from Phabricator. Differential Revision: D72862531
This pull request has been merged in facebook/Ax@1822b916f4f661f9be411bf6d3e2ce4a6eb0bbb8.