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

Document hawk/dove convergence logic + collect model status (running/convergence) #65

Merged
merged 4 commits into from
Feb 21, 2024

Conversation

rlskoeser
Copy link
Contributor

  • include write up of convergence logic in hawk/dove readme
  • include status (running/converged) in model data collection so we can make sure we're analyzing runs where the model stabilized
  • add an option to limit total batch runs for convenience
  • make agent data collection optional in batch run script

@LaraBuchak : please review the write up of the convergence logic

  • is it clear enough?
  • are you ok with these numbers (30 and 15 for rolling window and minimum window)?

here's a more readable version of the convergence logic in the hawk/dove model readme: https://github.com/Princeton-CDH/simulating-risk/tree/doc-convergence/simulatingrisk/hawkdove#convergence

@rlskoeser rlskoeser merged commit 3e24e0b into main Feb 21, 2024
3 of 4 checks passed
@rlskoeser rlskoeser deleted the doc-convergence branch February 21, 2024 20:42
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.

1 participant