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

Aberta
  • Algo errado?
#2,222 3 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
52/100
Tipo de issue
Documentação
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
c
Domínio
tooling

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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)

Linguagem predominante
C
Estrelas
3
Forks
7
Merge médio
6h 32min
PRs com merge (30d)
285

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de InauguralSystems/EigenScript

Todas as issues de InauguralSystems/EigenScript

Issues semelhantes

Mais issues de C

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.