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

オープン 初心者向け
#5,752 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1〜3時間
初心者へのやさしさ
85/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
markdown
領域
documentation

調査の方向性

workshop/side-quest-23-01-ab-roundrobin.md を開き、タイブレークの文、チェックポイント、"Predict assignment order" テーブルを読み、それらをリンクされた統計的バランシングのドキュメントと比較してください。first-in-array の主張をランダムなタイブレークに置き換え(least-used ルールは維持)、runs 1/2/4/5 を「同点のバリアントのうちの1つ」として書き直し、状態ブランチを experiments/{sanitizedWorkflowID} に修正しつつ、storage、weight、start_date/end_date に注意してください。ファイル内のすべての主張が文書化された動作と一致したら完了です。

索引モデルが issue の本文から書いたものです。

説明

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
主要言語
JavaScript
スター
62
フォーク
43
平均マージ
11時間 26分
マージ済み PR(30日)
13

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

  • Dockerfile・Docker Compose ファイルなし
  • プルリクエストのテンプレートなし
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

githubnext/gh-aw-workshop のほかの issue

githubnext/gh-aw-workshop の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。