logaritmisk
  • Joined on 2023-10-06
logaritmisk commented on issue logaritmisk/trueskill-tt#17 2026-08-27 16:02:12 +00:00
Per-slice SkillStore is dense over the global index space — memory scales with the largest competitor index, not slice size

Fixed in 6d2573b. Both fixes you proposed, together — compact storage and the resolved slot on Item, because either alone is insufficient: compact storage without the cached slot just moves…

logaritmisk commented on issue logaritmisk/trueskill-tt#18 2026-08-27 15:54:02 +00:00
Input validation is debug-only, and the History ingestion path bypasses InferenceError entirely

One more item enforced in release — 1ac3b21 — but staying open, and the remainder needs a decision from you rather than more work from me.

**Done: EventBuilder::weights (`src/event_build…

logaritmisk closed issue logaritmisk/trueskill-tt#23 2026-08-27 15:50:58 +00:00
Dead and duplicated code: a stale iterate_to_convergence, a copy-pasted sweep, and #[allow(dead_code)] used as a mute button
logaritmisk commented on issue logaritmisk/trueskill-tt#23 2026-08-27 15:50:55 +00:00
Dead and duplicated code: a stale iterate_to_convergence, a copy-pasted sweep, and #[allow(dead_code)] used as a mute button

The remaining five are done — 06ed24b, aff3fb9, 4e04336. Closing.

logaritmisk closed issue logaritmisk/trueskill-tt#25 2026-08-27 15:43:54 +00:00
cargo publish would fail: no description, license, or repository — and the crate has no top-level docs
logaritmisk commented on issue logaritmisk/trueskill-tt#25 2026-08-27 15:43:51 +00:00
cargo publish would fail: no description, license, or repository — and the crate has no top-level docs

All five acceptance criteria met. trueskill-tt 0.2.0 is published to the kellnr registry.

logaritmisk pushed to main at logaritmisk/trueskill-tt 2026-08-27 15:37:44 +00:00
07285283b6 chore: Release trueskill-tt version 0.2.0
b73cf0145a chore: dual-license MIT OR Apache-2.0
56ff01074f docs(cargo): correct the licence note — kellnr does not require one
8d47e54a8a chore: keep the 48 MB ATP dataset out of the published crate
7de092ba12 chore: target releases at the private kellnr registry
Compare 13 commits »
logaritmisk pushed tag v0.2.0 to logaritmisk/trueskill-tt 2026-08-27 15:37:44 +00:00
logaritmisk commented on issue logaritmisk/trueskill-tt#23 2026-08-27 15:22:20 +00:00
Dead and duplicated code: a stale iterate_to_convergence, a copy-pasted sweep, and #[allow(dead_code)] used as a mute button

One more item off this list in eeb43e3: color_greedy carried #[allow(dead_code)] while being called by recompute_color_groups — a mute button on a live function, which is the specific…

logaritmisk closed issue logaritmisk/trueskill-tt#28 2026-08-27 15:22:14 +00:00
log_evidence_internal's forward flag is only a filtering quantity before convergence
logaritmisk closed issue logaritmisk/trueskill-tt#31 2026-08-27 15:22:09 +00:00
Skill's compile-time divergence tripwire is only half-enforced
logaritmisk closed issue logaritmisk/trueskill-tt#30 2026-08-27 15:22:05 +00:00
Spec's tolerance-not-bit-identity caveat for filtered estimates is conservative
logaritmisk closed issue logaritmisk/trueskill-tt#29 2026-08-27 15:22:02 +00:00
log_evidence takes &mut self but mutates nothing
logaritmisk closed issue logaritmisk/trueskill-tt#27 2026-08-27 15:21:59 +00:00
converge() panics on a History with no events (usize underflow in iteration())
logaritmisk commented on issue logaritmisk/trueskill-tt#30 2026-08-27 15:21:56 +00:00
Spec's tolerance-not-bit-identity caveat for filtered estimates is conservative

Spec amended in eeb43e3docs/superpowers/specs/2026-08-27-filtered-estimates-design.md, Risks.

The caveat is kept (it is not wrong) with the measurement recorded beside it: forcing the…

logaritmisk commented on issue logaritmisk/trueskill-tt#31 2026-08-27 15:21:48 +00:00
Skill's compile-time divergence tripwire is only half-enforced

Fixed in eeb43e3. TimeSlice::add_events now spells out all four Skill fields instead of ..Default::default(), so a new field is a compile error at both construction sites rather than only…

logaritmisk commented on issue logaritmisk/trueskill-tt#28 2026-08-27 15:21:43 +00:00
log_evidence_internal's forward flag is only a filtering quantity before convergence

Taken the "document the constraint" branch in eeb43e3 rather than deleting the flag.

log_evidence_internal now carries:

forward selects skill.forward as each event's prior instead of…

logaritmisk commented on issue logaritmisk/trueskill-tt#29 2026-08-27 15:21:35 +00:00
log_evidence takes &mut self but mutates nothing

Fixed in eeb43e3. log_evidence, log_evidence_for and log_evidence_internal now take &self.

It was not the mechanical change it looked like. Under the rayon feature the closure in…

logaritmisk commented on issue logaritmisk/trueskill-tt#27 2026-08-27 15:21:26 +00:00
converge() panics on a History with no events (usize underflow in iteration())

Already fixed — but nothing pinned it, so it does now (eeb43e3).

The guard landed in f4e2922, and git merge-base --is-ancestor 7742b2b f4e2922 confirms the rev you filed against predates…

logaritmisk closed issue logaritmisk/trueskill-tt#19 2026-08-27 15:12:52 +00:00
HistoryBuilder::online(true) is non-functional — Skill.online is never written