-
Notifications
You must be signed in to change notification settings - Fork 263
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
Adding documentation for result set #660
Conversation
45fcdf4
to
734e512
Compare
Maybe consider using |
Good shout. Will do that. |
each repetition:: | ||
|
||
>>> pprint.pprint(results.payoffs) | ||
[[[3.0, 3.0, 3.0, 3.0, 3.0, 3.0], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will actually need to be changed once @Nikoleta-v3 #635 goes in. Ideally that should go first and then it'll be straightforward to fix this.
Basically: we're currently counting self interactions twice for certain things. This has been fixed in #635.
Working on this now that #635 is in. |
@@ -12,7 +12,7 @@ probability:: | |||
|
|||
|
|||
We can view the results in a similar way as for described in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo here. "as for" should be "as"
The tournament no longer has an interactions attribute.
Thanks @meatballs, have fixed 👍 |
I think you've both ok'd this so going to merge. Hope that's ok. |
Closes #652.
The extra details (looking at a the first item in a list whilst the whole list is skipped) for some of the doctests are just so that an error would be caught if the result set changed. Can't test the whole list because of floating points (rounding didn't seem to do the trick).