-
Notifications
You must be signed in to change notification settings - Fork 264
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
Write a manage.py script #32
Comments
I'm going to incorporate this with a new
|
At first glance, caching would be the first things to do to boost performance. |
See #41 for caching for pairs of deterministic strategies. |
Absolutely huge pull request! Amazing! Awesome @langner ! I'll leave this issue open as I still want to parallelise when I get a chance. That does need the overall better |
(I am learning a lot of Python from your pull request @langner : very much appreciated). |
Can this one be closed now? |
I think I'd still like a manage.py script, which we don't have so that we could move the functionality of
actually that's the only extra thing I'd want in there that I can think off.. Perhaps not needed? |
why not just add any additional functionality as methods in the TournamentManager class? Makes them much easier to test. |
Does it make sense to have On Mon, Mar 23, 2015 at 8:33 AM Owen Campbell notifications@github.com
|
Probably doesn't make much sense. Anyone who knows enough to want to run the tests is going to be perfectly capable of typing the command to do so. Running tests from within TournamentManager would actually require more typing! |
Yeah... Cool. Let's scrap this. On Mon, Mar 23, 2015 at 10:47 AM Owen Campbell notifications@github.com
|
As the number of strategies grows: the mathces (or just the random repetitions) can be parallelised (possibly even making use of caching for non stochastic strategies...).
The text was updated successfully, but these errors were encountered: