Issue hygiene: #1682 duplicates half of #1646, and #2212 plus #2198's open boxes are already in #1688

Đang mở
  • Có gì không ổn?
#2,222 3 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
52/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ệ
c
Lĩnh vực
tooling

Hướng nghiên cứu

Start by reading the linked issues #1646, #1682, #1688, #2198, and #2212, especially their Done-when boxes and comments. Compare the audit claims with the referenced locations in lib/ui_w_data.eigs, src/eigenscript.h, src/tape_read.c, src/state.c, and src/vm.h; then record which items are confirmed or refuted here. Done when the duplicate ownership and remaining task_sched_trace references are resolved in the tracker.

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

Mô tả

area:gfx area:observer found-by:duet-audit kind:tracking

Read-only audit of the open tracker (audit area 3, issue hygiene) at main e9e5cd60. Two groups of open issues track the same defect under different numbers, so whoever fixes one may not close the other. Nothing was closed or edited by the audit.

1. #1682 is #1646's second defect
  • #1646 (2026-10-06), section "Second defect: selection contrast": lib/ui_w_data.eigs fills a highlighted ASCII cell with the highlight colour, then draws the character in hex_ascii/text_dim, which is close to it, so "the selected text ("EigenScript hex_view" in the screenshot) is unreadable". Its Done-when box 2: selected ASCII cells draw text in a contrasting colour, with a test.
  • #1682 (2026-10-08): selecting "EigenScript hex_view" in examples/ui_hex.eigs shows "a solid highlight bar with no characters". Done-when: selected ASCII characters stay readable; a draw-list or pixel assertion covers a selected ASCII cell.

Same example, same selection, same fix. Still present on main: lib/ui_w_data.eigs:676 fills the cell with the highlight colour hc, and :688 draws the glyph with ascol, with no selected-text colour.

2. #2212 and #2198 (boxes 2 and 3) are inside #1688
  • #1688 asks to delete the machinery left unreachable by #1677: obs_cfg_sync, the O cfg / O win emitters and readers, trace_obs_window_binding, observer_slot_set_window, and the task.c scheduler-trace helpers, with replay refusing old tapes. Its 2026-10-21 and 2026-10-24 comments list the stale source comments, including src/eigenscript.h "set via set_observer_thresholds" and src/tape_read.c:173-174 naming builtin_set_observer_scale/builtin_set_observer_thresholds.
  • #2212 (2026-10-30) asks to fix exactly those two comments (src/eigenscript.h:2461 and src/tape_read.c:173-174 on main).
  • #2198's open boxes 2 and 3 ask for a decision on whether replay refuses an O win record and removes trace_obs_window_binding / win_override. #1688's Done-when already chooses: delete the readers and refuse old-format tapes.
  • #1688 was waiting for "the tape-bump release". Its 2026-10-24 comment notes the bump to format 7 shipped without the removal; main is now at format 8 (src/trace.h:32), and every function it names is still defined (src/trace.c:1572 obs_cfg_sync, src/trace.c:1602 trace_obs_window_binding, src/eigenscript.c:1269 observer_slot_set_window, src/task.c:307/324/473).

Not in any of the three: src/state.c:310 ("task_sched_trace of 1 arms it from a program") and src/vm.h:691 ("builtins.c task_sched_trace") still describe the removed builtin as live. Both are listed in #1688's 2026-10-24 comment.

Expected vs actual
  • Expected: one open issue per defect, so the fix's Closes line retires all of it.
  • Actual: two numbers for the hex-view contrast bug, and three for the observer-config removal, with #2198 asking a question #1688 already answers.

Done when

  • #1682 is closed as a duplicate of #1646 (or #1646's box 2 is moved to #1682 and #1646 says so), so exactly one open issue owns the selected-ASCII contrast fix.
  • #2212 and #2198's boxes 2 and 3 are folded into #1688 (closed as duplicates with a pointer, or #1688 links them as sub-issues), and #1688's Done-when no longer says "in the tape-bump release" now that formats 7 and 8 have shipped.
  • Whichever issue owns the comments lists src/state.c:310 and src/vm.h:691, and its grep box covers task_sched_trace.
  • Each item above is marked confirmed or refuted in a comment here.

— Duet (orchestrator)

Ngôn ngữ chính
C
Star
3
Fork
7
Merge trung bình
6 giờ 32 phút
Pull request đã merge (30 ngày)
285

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.

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 InauguralSystems/EigenScript

Tất cả issue của InauguralSystems/EigenScript

Issue tương tự

Thêm issue về C

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.