Skip to content

Conversation

@meatballs
Copy link
Member

Fixes #248

docs/usage.rst Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Could you add a quick sentence saying something like 'which shows that Cooperator had 0 wins and Defector 80 wins' (just to point at the order that we get out)?

@marcharper
Copy link
Member

Cool! Does this aggregate the wins over tournament repetitions? Just checking if we can make a nice plot of the distribution of wins (like the mean payoff plot).

@meatballs
Copy link
Member Author

At the moment, it aggregates, but I can easily split it out and show the wins per repetition, if that's useful.

@drvinceknight
Copy link
Member

At the moment, it aggregates, but I can easily split it out and show the wins per repetition, if that's useful.

I think in the interest of not losing data, this is a good shout. You could have a get_mean_wins_per_repetition method to complement that (or something less horribly verbose)?

Copy link
Member

Choose a reason for hiding this comment

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

"winsin" -> "wins in"
"wins in " -> "wins in"

@drvinceknight
Copy link
Member

This looks great to me! 👍

marcharper added a commit that referenced this pull request Oct 1, 2015
@marcharper marcharper merged commit f4b6a93 into master Oct 1, 2015
marcharper added a commit to marcharper/Axelrod that referenced this pull request Nov 2, 2015
@meatballs meatballs deleted the 248 branch November 8, 2015 15:16
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.

4 participants