Skip to content
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

Introducing meta players #126

Merged
merged 4 commits into from
Mar 26, 2015
Merged

Conversation

langner
Copy link
Member

@langner langner commented Mar 25, 2015

This abstracts the core of MetaHunter into an abstract MetaPlayer and adds some more involved metaplayers. Depending on the interpretation, these meta players could be regarded as cheaters, since they load other strategies at runtime. They do not, however, modify strategies or run separate tournaments, just record the running scores these strategies would have had in the current game. In princple, the same could be written explicitely without importing the strategies (alhtough it would be much longer). Anyway, these players are not particularly good (I though MetaWinner would do somewhat better, though).

@drvinceknight
Copy link
Member

Yeah, I agree: I don't think these count as cheaters. They're just taking very detailed notes throughout the tournament.

@langner
Copy link
Member Author

langner commented Mar 25, 2015

They also boost the walltime of the tournament... creating more optimization challenges! :)

@drvinceknight
Copy link
Member

Have you seen #125 ? I seem to be having problems with the parallelised version of the tournament now...

@drvinceknight
Copy link
Member

(That's not directly relevant to this PR but just pinging you in case you have the same problem. Heading out the door now but will take a look at this shortly.)

@drvinceknight
Copy link
Member

This is looking nice.

How much longer is the run time?

Also, could you tweak the contribution guidelines in the docs for the change from __init__ to _strategies.

@langner
Copy link
Member Author

langner commented Mar 26, 2015

The whole thing now runs in about 100s on my machine.

drvinceknight added a commit that referenced this pull request Mar 26, 2015
Introducing meta players
@drvinceknight drvinceknight merged commit 0bc2290 into Axelrod-Python:master Mar 26, 2015
marcharper pushed a commit to marcharper/Axelrod that referenced this pull request Nov 2, 2015
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.

2 participants