Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

probation_start is set while the criterion is still failing

Aperta
  • Qualcosa non va?
#1,844 0 commenti 1 reazione 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
53/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python, sql
Ambito
backend-api-design

Direzione di ricerca

Start with state_machine._probation_start and the outlived_grace branch in transition(), which the issue identifies as stamping probation_start; trace how confirmed status changes from fail to pass. Check the existing backfill approach for rows stamped mid-failure. Done means probation starts only on recovery, the read-side compensations can be removed, and existing affected rows are corrected.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

probation_start is stamped as soon as a failure is confirmed, then re-stamped to "tomorrow" on every failing night after that. So a set probation_start does not mean the criterion is serving probation — it may still be failing.

Every reader has to compensate with the same extra check:

Reader Compensation
is_serving_probation (API) confirmed_status == 'pass'
feed_search.sql confirmed_status = 'pass'
roll_up_on_probation cancels on any confirmed failure
state_machine.phase() returns ON_PROBATION for both, leaves it to the caller

Recent fixes have all been on the read side: #1840, #1841, #1842, #1843.

Fix

Stamp probation_start only on the recovery — the run where confirmed_status goes fail → pass. last_confirmed_failure_at already records that a failure happened, so the "stamp tomorrow on every failing night" trick is no longer needed to land the start on the repair day.

Then probation_start is not None means serving probation, and the four compensations above come out.

  • state_machine._probation_start, plus the outlived_grace branch in transition(), which stamps too
  • backfill for rows currently stamped mid-failure
Lingua principale
Python
Stelle
34
Fork
7
Merge medio
2g 4h
PR unite (30g)
8

Preparare l'ambiente

  • Include un Dockerfile o un file Docker Compose
  • Ha un modello di pull request
  • Nessuna guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di MobilityData/mobility-feed-api

Tutte le issue di MobilityData/mobility-feed-api

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.