[workshop-sync] workshop/side-quest-23-01-ab-roundrobin.md: tie-break and state-storage description is outdated

Aperta Adatta ai principianti
  • Qualcosa non va?
#5,752 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
1-3 ore
Idoneità per principianti
85/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
markdown
Ambito
documentation

Direzione di ricerca

Apri workshop/side-quest-23-01-ab-roundrobin.md e leggi la frase di tie-break, il checkpoint e la tabella "Predict assignment order", poi confrontali con i documenti di bilanciamento statistico collegati. Sostituisci le affermazioni first-in-array con un tie-breaking casuale (mantenendo la regola least-used), riscrivi le esecuzioni 1/2/4/5 come "una delle varianti in parità" e correggi il branch di stato su experiments/{sanitizedWorkflowID} annotando storage, weight e start_date/end_date. Fatto quando ogni affermazione nel file corrisponde al comportamento documentato.

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

Descrizione

documentation

Workshop file reviewed

workshop/side-quest-23-01-ab-roundrobin.md (gh-aw v1.2.2)

Problem

  1. The file says: "Picks the variant with the lowest invocation count (ties are broken by first-in-array order)." and "Because ties are broken by first-in-array order, you can predict every assignment before you run the workflow". The checkpoint repeats "I know ties are broken by first-in-array order". The "Predict assignment order" section and its table (runs 1, 2, 4, 5) depend on this.
  2. The file says: "Loads state from experiments/{workflow-id} (created on first run)." The branch name is actually experiments/{sanitizedWorkflowID} (lowercased, hyphens removed, e.g. my-workflow -> experiments/myworkflow). Also storage: repo (default) vs cache is not mentioned. The file also omits the weight option (weighted random selection) and start_date/end_date (returns the control variant without incrementing counters).

Current correct behaviour

From https://github.github.com/gh-aw/experimental/experiments/#statistical-balancing: "By default, gh-aw chooses the least-used variant on each run. If multiple variants are tied, including on the first run, one of them is chosen at random." With weight, selection is weighted random.

Suggested fix

  • Replace the first-in-array tie-break claims with random tie-breaking, including in the checkpoint.
  • Rework "Predict assignment order" and the table: the least-used rule still holds (e.g. a newly added variant with count 0 is picked next), but ties among equal counts are not predictable, so runs 1, 2, 4 and 5 should show "one of the tied variants".
  • Correct the state branch name to experiments/{sanitizedWorkflowID} and mention the storage setting.

Generated by 🔍 Workshop Sync Check · copilot · auto · ~21 AIC · ⌖ ~1 AIC · ⊞ 4.7K · ◷

  • expires on Nov 16, 2026, 10:03 AM UTC
Lingua principale
JavaScript
Stelle
62
Fork
43
Merge medio
11h 26m
PR unite (30g)
13

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

  • Nessun Dockerfile né file Docker Compose
  • Nessun 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 githubnext/gh-aw-workshop

Tutte le issue di githubnext/gh-aw-workshop

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.