search

Yelp / MOE

A global, black box optimization engine for real world metric optimization.
Other
1.31k stars 140 forks source link

Merge similar optimization methods in C++ #80

Open suntzu86 opened 10 years ago

suntzu86 commented 10 years ago

BLOCKED ON: #274

Currently I have things like: ComputeOptimalPointToSampleWithRandomStarts and ComputeOptimalPointToSampleViaMultistartGradientDescent

The only difference here is that the second one lets you specify the starting_points for MGD and the first one generates them for you. The first one wraps the second one. That's a lot of duplicated comments and junk for such a teeny tiny difference.

suntzu86 commented 10 years ago

This would be better solved by something like #274. Having gigantic argument lists where half of them can be null to engage some default behavior is worse than naming these 'endpoints' separately.