Skip to content

fix(sim-core): remove labor fill bias and market-order nondeterminism#4

Merged
JoeHowarth merged 5 commits into
mainfrom
soa-fixes-followup
Feb 27, 2026
Merged

fix(sim-core): remove labor fill bias and market-order nondeterminism#4
JoeHowarth merged 5 commits into
mainfrom
soa-fixes-followup

Conversation

@JoeHowarth

Copy link
Copy Markdown
Owner

Summary

  • make labor assignment matching explicitly discrete and deterministic under partial fills by selecting unit winners using fill quantity and deterministic tie-breakers instead of zip pairing
  • sort settlement merchant participation order before generating market orders so order IDs and tie-break outcomes are stable across runs with the same seed
  • fix MerchantAgent::can_stockpile_at to reflect owned facility presence at a settlement and add unit test coverage

Validation

  • cargo test -q

@JoeHowarth JoeHowarth merged commit d129a2d into main Feb 27, 2026
2 checks passed
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