Skip to content

Short and Long Run-time strategies #771

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

Closed
marcharper opened this issue Nov 23, 2016 · 1 comment
Closed

Short and Long Run-time strategies #771

marcharper opened this issue Nov 23, 2016 · 1 comment

Comments

@marcharper
Copy link
Member

I'd like to add a few more meta strategies to the library, such as MetaWinners for the collections of stochastic and deterministic strategies. I do not, however, want to add more long run time strategies, since we've already got several and they take up the vast majority of the run time (especially against each other).

So I'd like to propose that we leave out the long run time strategies from the default collection of strategies, or at least add a prominent collection of "short run time" strategies:

fast_strategies = [s for s in all_strategies if obey_axelrod(s())
                           and not s().classifier['long_run_time']]

This is because I don't think new comers to the library will necessarily be aware that there are long time strategies that take up a lot of run time.

@drvinceknight
Copy link
Member

My preference would be to add short_run_time. I think leaving the
strategies as is is important to do.

If only for backwards compatibility, but also, the risk of newcomers not
knowing that there's another set of valid strategies elsewhere is in my
opinion worse than that they might have some tournaments that run for a
while. The status quo of: all 'valid' strategies are in strategies and
they can be further obtained using the filters is a good way of doing
things.

I think that in practice, if anyone were to run something on the full set
of strategies I think they would expect a long time anyway (and they would
perhaps have enough experience to deal with the filtering).

But I think having a helper list of short_run_time strategies is a good
idea. We always said that we could keep adding these shortcut lists :)

On Wed, 23 Nov 2016, 15:43 Marc Harper, [email protected] wrote:

I'd like to add a few more meta strategies to the library, such as
MetaWinners for the collections of stochastic and deterministic strategies.
I do not, however, want to add more long run time strategies, since we've
already got several and they take up the vast majority of the run time
(especially against each other).

So I'd like to propose that we leave out the long run time strategies from
the default collection of strategies, or at least add a prominent
collection of "short run time" strategies:

fast_strategies = [s for s in all_strategies if obey_axelrod(s())
and not s().classifier['long_run_time']]

This is because I don't think new comers to the library will necessarily
be aware that there are long time strategies that take up a lot of run time.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#771, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCGWkZd_j3EV6MrHhORGO9TR24cPgIiks5rBF8fgaJpZM4K6vNc
.

drvinceknight added a commit that referenced this issue Dec 19, 2016
Closes #771

Have included a very basic test and have also done some minor tidying
up.
drvinceknight added a commit that referenced this issue Dec 19, 2016
Closes #771

Have included a very basic test and have also done some minor tidying
up.
drvinceknight added a commit that referenced this issue Dec 19, 2016
Closes #771

Have included a very basic test and have also done some minor tidying
up.
drvinceknight added a commit that referenced this issue Dec 22, 2016
Closes #771

Have included a very basic test and have also done some minor tidying
up.
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

No branches or pull requests

2 participants