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

hawk/dove: add option to adjust risk attitude based on recent or total payoff #54

Merged
merged 4 commits into from
Dec 15, 2023

Conversation

rlskoeser
Copy link
Contributor

@rlskoeser rlskoeser commented Dec 14, 2023

modify hawk/dove multiple risk attitudes simulation with parameter to control whether agents compare against neighbors' cumulative payoff or payoff since the last adjustment round

Copy link

codecov bot commented Dec 14, 2023

Codecov Report

Merging #54 (add8299) into main (3ca748e) will increase coverage by 9.39%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   51.61%   61.01%   +9.39%     
==========================================
  Files          15       16       +1     
  Lines         618      767     +149     
==========================================
+ Hits          319      468     +149     
  Misses        299      299              

@rlskoeser rlskoeser temporarily deployed to qa December 14, 2023 19:43 Inactive
@rlskoeser rlskoeser temporarily deployed to qa December 14, 2023 19:43 Inactive
Copy link

@quadrismegistus quadrismegistus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! And a ton of testing work. How long did those take to write?

@rlskoeser
Copy link
Contributor Author

LGTM! And a ton of testing work. How long did those take to write?

I wrote them before and forgot to add to git! 🤦‍♀️ Only figured that out when I went to add the revisions to the tests for the changes and discovered the file wasn't in version control.

@rlskoeser rlskoeser merged commit 415ea08 into main Dec 15, 2023
4 checks passed
@rlskoeser rlskoeser deleted the hawkdove-adjust-recent-payoff branch December 15, 2023 17:51
rlskoeser added a commit that referenced this pull request Feb 12, 2024
hawk/dove: add option to adjust risk attitude based on recent or total payoff
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.

None yet

2 participants