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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#5,752 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
1/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
85/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
markdown
Área
documentation

Línea de trabajo

Abre workshop/side-quest-23-01-ab-roundrobin.md y lee la frase de desempate, el checkpoint y la tabla "Predict assignment order"; luego compáralos con los documentos de equilibrio estadístico enlazados. Sustituye las afirmaciones de first-in-array por desempate aleatorio (manteniendo la regla de least-used), reescribe las ejecuciones 1/2/4/5 como "una de las variantes empatadas" y corrige la rama de estado a experiments/{sanitizedWorkflowID}, teniendo en cuenta storage, weight y start_date/end_date. Terminado cuando cada afirmación del archivo coincida con el comportamiento documentado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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
Lenguaje dominante
JavaScript
Estrellas
62
Forks
43
Merge medio
11 h 26 min
PR fusionados (30 d)
13

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

  • Sin Dockerfile ni archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de githubnext/gh-aw-workshop

Todos los issues de githubnext/gh-aw-workshop

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.