-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add run_with_required_agents function for TryCP scenarios #115
Conversation
e0f788a
to
701045f
Compare
@ThetaSinner, as per our discussion; I changed this to only be an option for TryCP scenarios instead of all scenarios. I'll drop the first two commits before merging to |
701045f
to
33fd2a6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the test time question, this looks good to me 👍
This calls `run` and checks that it completes with at least the passed number of agents. This can be overridden via the `MIN_REQUIRED_AGENTS` environment variable.
b6e589d
to
8851b54
Compare
18757af
to
ee58fec
Compare
The minimum number of required agents to run the scenario to completion. Can be overridden via an environment variable
MIN_REQUIRED_AGENTS
.