-
Notifications
You must be signed in to change notification settings - Fork 268
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
Add long run time to player classifier #690
Comments
We should define long-run strategies somehow. I suggest we pick one strategy as the threshold, and I nominate PSO Gambler. |
Haha: I literally just pushed this using the same classification we had already in place: the meta ones. Easy enough to change though: I'm happy with whatever threshold. |
That's where it is already. |
Ok cool :) If we wanted to change it, we just need to change this test and then hunt them down: /~https://github.com/Axelrod-Python/Axelrod/blob/master/axelrod/tests/unit/test_classification.py#L168 |
See chat on #689, idea is to add
long_run_time
to the classifier dict. We could then change:to be
This could be
False
by default so we'd only need to actually add it to thelong_run_time_strategies
. All the tests are in place that should pick it up for the lists and we could add a test to the global test player class too...The text was updated successfully, but these errors were encountered: