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

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#5,752 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
85/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
markdown
Lĩnh vực
documentation

Hướng nghiên cứu

Mở workshop/side-quest-23-01-ab-roundrobin.md và đọc câu tie-break, checkpoint, và bảng "Predict assignment order", sau đó so sánh chúng với các tài liệu cân bằng thống kê được liên kết. Thay các khẳng định first-in-array bằng tie-breaking ngẫu nhiên (giữ nguyên quy tắc least-used), viết lại runs 1/2/4/5 thành "một trong các biến thể hòa nhau", và sửa nhánh trạng thái thành experiments/{sanitizedWorkflowID} đồng thời lưu ý storage, weight, và start_date/end_date. Hoàn tất khi mọi khẳng định trong tệp khớp với hành vi đã được tài liệu hóa.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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
Ngôn ngữ chính
JavaScript
Star
62
Fork
43
Merge trung bình
11 giờ 26 phút
Pull request đã merge (30 ngày)
13

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

  • Không có Dockerfile hay tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của githubnext/gh-aw-workshop

Tất cả issue của githubnext/gh-aw-workshop

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.