Added Bandit UCB1-tuned implementation

Yelp / MOE

A global, black box optimization engine for real world metric optimization.

Other

1.31k stars 139 forks source link

Closed norases closed 10 years ago

norases commented 10 years ago

***** PEOPLE ***** Primary reviewer: @sc932

Reviewers: @suntzu86

***** DESCRIPTION ** Branch Name: norases_366_ucb1_tuned_bandit Ticket(s)/Issue(s): #366

***** TESTING DONE ***** make test make style-test

sc932 commented 10 years ago

A few very minor nits, but looks great.

Excellent job getting these all out so quickly!

Fix and ship :)

suntzu86 commented 10 years ago

looks good. My comments are mostly style but I have one organizational concern about how we treat variance. @sc932 opinion appreciated, see above.

Also, pull master. Your test failure should've been fixed by @bulutcocuk's latest push.

suntzu86 commented 10 years ago

shipit

sc932 commented 10 years ago

:ship: :boat: :goat: