-
Notifications
You must be signed in to change notification settings - Fork 944
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
Batch runner redux #871
Batch runner redux #871
Conversation
Intro Tutorial fixes
…_parameters Fix empty/None `variable_parameters` argument to BatchRunner (#861)
BatchRunnerMP ---- - removed pathos (unneccessary as unordermap in pathos is same as multiprocessing) - refactored so compatible with Windows, Mac and Linux OS Batchrunner ---- - add two functions - get_collector_model: returns dictionary of dataframes for datacollector model reporters - get collector_agents: returns dictionary of dataframes for datacollector agent reporters (key for dictionaries is unique parameter variation with iteration number) DataCollector ---- - expanded to allow more collection types to be passed in model reporter (agents are handled internally). This was necessary to becuase lambda cannot be used at the mudel level for multiprocessing key_name: <class attribute> key_name: function (e.g schedule.get_agent_count) key_name: lambda function key_name: [function, [<arguments for function>]
With Documentation Update
Closing to fix lack of codecov report |
Sorry for not reviewing this already. Is there a new PR coming? Also: There seems to be quite a lot unrelated file changes. Could you somehow separate the batchrunner changes from the rest? Also also: Thank you so much already for doing this. I know this must have been quite a lot of work and it is very essential to fix batchrunner! |
@Corvince Thanks! As you pointed out there really is two areas of fixes - (1) Batchrunner the which includes includes changes to datacollection because of their interdependencies, a batchrunnerMP test module, and documentation update. (2) The example folders, these were all failing to run because of import path issues. Which I thought was weird so I may be misunderstanding something, but I got them running to make sure all the travis checks pass, but then Travis did not run and I did not get a codecov report. I am trying figure out why, but I will try to seperate them and get the travis check and codecov working for cleaner pull requests. |
Summary of Changes:
I recommend after anticipated further modifications we do a version push. My timing on this is to give you time to review and then discuss at next Dev Session 6 August.
BatchRunnerMP
Batchrunner
(key for dictionaries is unique parameter variation with iteration number)
DataCollector
expanded to allow more collection types to be passed in model reporter (agents are handled internally so I did not think it was needed). This was necessary because lambda cannot be used at the model level for multiprocessing
Tutorial