Winnie/bankwars
Watch
1
0
Fork
You've already forked bankwars
 0
Code Issues 9 Pull requests 2 Projects Releases 6 Packages Wiki Activity Actions

no post churn display #63

New issue
Closed
opened 2026-09-29 20:19:11 +00:00 by robertismo · 2 comments
robertismo commented 2026-09-29 20:19:11 +00:00
Copy link

after setting your deposit rate the player should see by how much of their churn they captured((post-deposit - pre-deposit + churn)/churn).

after setting your deposit rate the player should see by how much of their churn they captured((post-deposit - pre-deposit + churn)/churn).
Winnie commented 2026-09-29 20:19:34 +00:00
Owner
Copy link

On it. Plan:

  1. Capture each bank's pre-flow deposits before depositFlow runs (the churn outflow is computed from them).
  2. In the deposit-phase display, after the flow, print for the human bank only: churn captured = (post − pre + churn) / churn as a percentage, where churn = free-float × pre-deposits. Zero-churn banks (free-float 0 or no deposits) skip the line to avoid a division by zero.
  3. go test . + build + an interactive smoke (scripted stdin, human seat) so the new line is actually exercised, then a PR against master.

Commenting again when the diff is up.

On it. Plan: 1. Capture each bank's pre-flow deposits before `depositFlow` runs (the churn outflow is computed from them). 2. In the deposit-phase display, after the flow, print for the human bank only: churn captured = (post − pre + churn) / churn as a percentage, where churn = free-float × pre-deposits. Zero-churn banks (free-float 0 or no deposits) skip the line to avoid a division by zero. 3. `go test .` + build + an interactive smoke (scripted stdin, human seat) so the new line is actually exercised, then a PR against master. Commenting again when the diff is up.
Winnie commented 2026-09-29 21:13:53 +00:00
Owner
Copy link

Shipped via PR #65: the human's deposit line now shows churn captured — (post − pre + churn) / churn as a percentage, zero-churn safe. Interactive smoke verified (104.6% hand-checked). Closing.

Shipped via PR #65: the human's deposit line now shows churn captured — (post − pre + churn) / churn as a percentage, zero-churn safe. Interactive smoke verified (104.6% hand-checked). Closing.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
158-market
157-control
156-records
155-stdlib
151-gauntlet
149-eligibility
147-report
145-vickrey
143-pauses
141-defaults
139-summary
137-version
127-minors
125-replay
126-negbuiltin
123-writeoff
124-revoke
121-panic
122-logf
120-finite
119-shoreup
116-acq-basis
88-failure-calc
113-player-display
merge-super
v050
canonical
105-bond-volume
106-bid-all
100-shelf-reset
100-nondet
exp13
98-tfix
94-amort
80-super-blend
89-t
91-nan
80-super-b
80-super-d
80-super-x4
80-super-x3
80-super-x1
87-let-if
80-super-x2
80-super-r1
80-super-ai2
80-super-ai2b
80-super-c
84-simult
80-super-ai
78-gaap
79-api-clean
75-reserve
73-acq-books
71-liquidity
69-bond-units
64-summary
67-loan-summary
63-churn-display
52-empire
53-mill
60-bench-roster
54-vault
55-adapt
39-main-overhaul
47-report
45-acct
46-iter-log
37-hybrids
40-version-harness
38/punishing-presets
41/fix-ci-checkout
v0.6.1
v0.6.0
v0.5.0
promotion-517
promotion-515
promotion-512
promotion-471
promotion-460
promotion-458
v0.4.0
v0.3.0
v0.2.0
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Winnie/bankwars#63
No description provided.