Skip to content

Refactor valuation marks out of economics#53

Merged
tradingexpert merged 1 commit into
mainfrom
feature/human-legibility-pass
May 20, 2026
Merged

Refactor valuation marks out of economics#53
tradingexpert merged 1 commit into
mainfrom
feature/human-legibility-pass

Conversation

@tradingexpert
Copy link
Copy Markdown
Owner

@tradingexpert tradingexpert commented May 20, 2026

Summary

  • Move valuation mark types and compact mark iteration into ordersim.valuation.
  • Keep top-level and ordersim.economics imports stable for existing users.
  • Align architecture, schema, economics docs, and agent module map with the new boundary.
  • Split valuation-specific tests out of economics tests.

Verification

  • python -m ruff check .
  • python -m pytest — 142 passed, total coverage 98.29%; ordersim.valuation 100%.
  • mkdir -p build/cpp && c++ -std=c++17 -Wall -Wextra -pedantic -Icpp cpp/tests/matching_engine_core_test.cpp -o build/cpp/matching_engine_core_test && build/cpp/matching_engine_core_test

@tradingexpert tradingexpert self-assigned this May 20, 2026
@tradingexpert tradingexpert merged commit 2d90ee4 into main May 20, 2026
8 checks passed
@tradingexpert tradingexpert deleted the feature/human-legibility-pass branch May 20, 2026 11:11
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