Adding UCB1 functionality. #11

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

wengzilla commented Aug 17, 2012

No description provided.

lib/bandit/players/ucb.rb
+ best = nil
+ best_rate = nil
+ experiment.alternatives.each { |alt|
+ rate = experiment.conversion_rate(alt) + confidence_interval
@bmuller

bmuller Nov 3, 2012

Owner

Shouldn't the line above include some params passed to confidence_interval?

@wengzilla

wengzilla Nov 5, 2012

Contributor

You're definitely right... I'll need to go in and fix that.

Owner

bmuller commented Feb 19, 2013

Oh - if you get a chance - a brief description in players.rdoc would be awesome.

Thanks!

Contributor

wengzilla commented Feb 20, 2013

Rebased and fixed as per your suggestions.

Owner

bmuller commented Feb 20, 2013

Merged in 2bb9096. Thanks!

@bmuller bmuller closed this Feb 20, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment