Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added multiple trial functionality, other minor changes #9

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Added multiple trial functionality, other minor changes #9

wants to merge 7 commits into from

Conversation

Chryzanthemum
Copy link

multiple_trials - iterates over an array and uses online trial for each pair of bandits/payouts
multiple_trials_bulk - sums up array of payouts to update choices, payouts without iterating
est_payouts - added option to choose which bandit to see payouts for
info - added number of bandits, number of wins for each bandit, and estimated payouts for each (wasn't sure why you wanted to put it under the bandit class)
current - added current bandit choice (choices[-1])

Benjamin Jiang added 3 commits April 10, 2018 17:37
slots/slots.py Outdated
'''
Feeds two arrays in and based on those results returns the next trial.
This really isn't optimized, there's a much better way of doing this if we don't
care about maintaining the

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maintaining the what?

Benjamin Jiang added 4 commits April 20, 2018 17:03
Some list/array comprehension stuff
So I combined my two multiple trial methods into a 'hard' and a 'lazy' option,  and I'm also starting to track each payout value. This definitely increases the size of the object, so maybe add an option to disable that, but it's important to the next step which is to add a time-sliding scale.
Multiple Trials condense, track all payout values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants