Skip to content

Keep recall-collapse baselines stable through silent days - #42

Closed
hamzamerzic wants to merge 2 commits into
mobius-os:mainfrom
hamzamerzic:fix/reflection-recall-baseline
Closed

Keep recall-collapse baselines stable through silent days#42
hamzamerzic wants to merge 2 commits into
mobius-os:mainfrom
hamzamerzic:fix/reflection-recall-baseline

Conversation

@hamzamerzic

Copy link
Copy Markdown
Contributor

Summary

  • keep silent days from rewriting an established recall baseline during a sustained outage
  • require three active historical days so one busy day cannot create a false baseline
  • cover prolonged collapse and sparse-history behavior with regression tests

Why

PR #38 introduced the recall-collapse signal using a trailing seven-day median. During a multi-day outage, silent days eventually drove that median below the alert threshold, so the outage became its own normal and the alert cleared while recall remained inactive.

Ignoring silent days alone would make a single busy day sufficient evidence. This follow-up uses a bounded 14-day nonzero window and requires three active days before treating the median as established.

Tests

  • PYTHONPATH=. pytest -q (71 tests, 2 subtests)

Co-authored-by: Möbius Agent <mobius-agent@users.noreply.github.com>
@hamzamerzic hamzamerzic added the bug Something isn't working label Aug 10, 2026
Co-authored-by: Möbius Agent <mobius-agent@users.noreply.github.com>
@hamzamerzic

Copy link
Copy Markdown
Contributor Author

Möbius agent review

Result: Blocking queue finding: do not merge this alongside #43. PR #43 replaces the sliding automatic verdict with a durable completion checkpoint and ordered recall evidence, which is the coherent solution to the same failure. Close #42 as superseded by #43.

I reread the full diff and checked the current GitHub workflow result; the reviewed revision is green unless the queue note above calls for a later rebase.

This is a disclosed agent review posted by the PR author’s account, not an independent maintainer approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant