-
Notifications
You must be signed in to change notification settings - Fork 267
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
Player is not evolvable. Use a subclass of EvolvablePlayer. #1345
Comments
Can you paste your code here? My guess is that you need to use the right type of mutation, only the "atomic" type should require an EvolveablePlayer. |
A few issues:
|
Hi Marc, Now i am getting ''TypeError: Player is not evolvable. Use a subclass of EvolvablePlayer.''. code: C = axl.Action.C mpE = axl.MoranProcess(playerswithevolable,mutation_method="atomic") |
That's what I expected. @drvinceknight any thoughts? Maybe we could have some kind of wrapper / transformer that makes any player generically evolvable in a trivial way for cases like this. |
Sorry for taking so long to get back to you on this. I like your suggestion @marcharper, a wrapper that allows player to evolve in a trivial sounds good. Would another approach be to add an evolve method to the base player class that does not do anything? |
Yes we could also do that, but then it removes the distinction between Player and EvolvablePlayer. Maybe that's a good idea but I'm not sure (for testing purposes at least). |
I hear you. It's two options, whatever your hunch tells you is fine by me. |
Post #1288 I'm thinking that it's better to make all Players evolvable, if only to remove multiple inheritance from the Player classes hierarchy. I'm not sure we actually gain much by having both |
Hi,
I would like to see how non-evolvable strategies behave in moran process compared to evolvable ones, however when i try to give players array in a mixed way i got this error. Is there a way to create such an environment?
many thanks
The text was updated successfully, but these errors were encountered: