Skip to content
This repository was archived by the owner on Jun 25, 2026. It is now read-only.

Support multiple leaderboard tables - #52

Open
KyleHerndon wants to merge 2 commits into
mainfrom
kyle/multi-leaderboard-tables
Open

Support multiple leaderboard tables#52
KyleHerndon wants to merge 2 commits into
mainfrom
kyle/multi-leaderboard-tables

Conversation

@KyleHerndon

Copy link
Copy Markdown
Contributor

Summary

  • add table-native rank_division responses with a TODO: delete rankings compatibility shim
  • let ruleset_strategy scoring config declare multiple leaderboard tables over arbitrary score keys, half-lives, and lookback windows
  • serialize primary table rankings for legacy callers while returning all configured tables

Validation

  • uv run pytest tests/test_commissioner_strategies.py -q
  • uv run python -m compileall commissioners tests/test_commissioner_strategies.py -q

Notes

  • No repo lint command is defined in pyproject/README/repo config.
  • Broader pytest runs reached the existing slower websocket/Docker coverage and hung in this environment after the strategy tests completed, so I stopped them.

Co-authored-by: GPT-5 Codex <codex@openai.com>
@KyleHerndon
KyleHerndon force-pushed the kyle/multi-leaderboard-tables branch 2 times, most recently from 99c9c34 to 2f9b959 Compare June 24, 2026 21:23
@KyleHerndon
KyleHerndon force-pushed the kyle/multi-leaderboard-tables branch from 2f9b959 to 5f8f2b4 Compare June 24, 2026 22:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant