Skip to content

Conversation

@gaffney2010
Copy link
Member

classifier_test in test_player was using dictionaries instead of the Classifiers class. This error was masked because all classifiers currently match the dictionaries. In the future, we will use formulas for some of the classifiers.

In addition to the changes on test_player, I removed some lines which altered expected classifiers to incorrect values. These lines accounted for the fact that the memory_depth was sometimes set on initializing. The new Classifiers paradigm can account for this.

@drvinceknight
Copy link
Member

Thanks @gaffney2010 👍

@drvinceknight drvinceknight merged commit 3635b70 into Axelrod-Python:master Apr 14, 2020
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.

3 participants