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 new way to calculate the probability an alternative is the winner #251

Closed
wants to merge 0 commits into from

Conversation

caser
Copy link
Contributor

@caser caser commented May 14, 2014

I added a new way to calculate the probability that a given alternative is the best. This is important for experiments with multiple alternatives, as the current z-score testing only compares the alternatives to the control and not to each other. The new feature simulates 10,000 draws from a probability distribution given each alternative's past performance, and then outputs the probability that it is the winner, given the rest of the alternatives.

@andrew
Copy link
Member

andrew commented May 18, 2014

This looks really interesting, I doubt I'll have time to properly review it until later in the week though.

One thing I did notice whilst having a quick browse was a file called changes.rtf, is that supposed to be in there, if so what's it for?

@caser
Copy link
Contributor Author

caser commented May 19, 2014

Okay, no worries on the time.

And good catch - that was a scratch file I was using for notes as I worked. I just pushed a new commit that no longer includes the file.

Cheers.

@caser
Copy link
Contributor Author

caser commented Sep 3, 2014

Hey - saw this hasn't been merged yet - would you like me to help walk you through the math & reasoning behind what I did?

@andrew
Copy link
Member

andrew commented Sep 20, 2014

@caser sorry for the delay, I'd like to get this merged in an released as 0.8.0, there are some merge conflicts at the moment, could you resolve those and then I can merge it in.

@caser
Copy link
Contributor Author

caser commented Sep 21, 2014

No worries - will look through it this week and get back to you.

@andrew
Copy link
Member

andrew commented Sep 24, 2014

We've move over here: #266

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