search

facebook / Ax

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

Delete some unneeded tests #2766

Closed esantorella closed 2 weeks ago

esantorella commented 2 weeks ago

Summary: Context: There used to be more BenchmarkProblem subclasses, and they used to implement their own 'repr' methods, so there were tests for the custom repr methods. Now BenchmarkProblem and its subclass SurrogateBenchmarkProblem get their __repr__ methods from being data classes. These tests have become annoying because they break with any change to `BenchmarkProblem, even if just changing the order of arguments.

This PR:

Reviewed By: Balandat

Differential Revision: D62518032

facebook-github-bot commented 2 weeks ago

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

facebook-github-bot commented 2 weeks ago

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