Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

`confidence` means two different things, and its type docs cite fields PT9 never sends

Aperta
  • Qualcosa non va?
#309 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
backend-api-design, documentation

Direzione di ricerca

Inizia da src/types/interlinearizer.d.ts:970 e dagli utilizzi dell’import PT9 in analysisMerger.ts:339 e :406, quindi esamina la discussione collegata sul significato di confidence in #254. Determina e documenta un significato per confidence, definisci cosa accade all’altro significato e aggiorna la documentazione del tipo affinché corrisponda ai campi PT9 distribuiti; considera la migrazione dei dati persistiti se cambia il significato del campo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Two problems in one field's story — one semantic, one documentation.

1. confidence is claimed twice

PT9 import spends AnalysisLink.confidence on anchor ambiguity: confidence: 'low' marks a record whose token placement was a guess among equal folded forms (analysisMerger.ts:339, :406).

#254 proposes spending it on approval provenance — stamping a bulk-approved link lower than a hand-approved one, so "approved" doesn't collapse "a human read this" with "a human swept a hundred of these" (#254's Confidence bullet, which relates it to #51 and #53, never to PT9). #51 is on hold as of 2026-08-25, so #254 is the live claimant.

Two different meanings for one per-assignment field, and neither issue records the collision. Decide which meaning confidence carries, and what mechanism the loser gets — a separate field, a flag on the link, or nothing.

Note the import use is already shipped and sitting in persisted data, so changing its meaning is a migration question, not only a naming one.

2. The type docs cite fields the format never sends

interlinearizer.d.ts:970 still says Paratext status / confidence are "inferred from InterlinearLexeme.IsGuess and .Score". The PT9 interlinear data we consume has neither — no IsGuess, no Score, only VerseData.Hash — and the shipped code means something else by both: status comes from Hash plus Excluded plus the merger's demotion rules, confidence from anchor ambiguity as above.

The doc points at a source that does not exist, for a field whose real meaning is question 1. Fix it once 1 is decided, rather than twice.

Size: S — a decision plus a doc fix; larger if the decision forces a new field.
Priority: P2 — nothing is broken for a user today, but #254 cannot stamp anything until this is settled.

Drafted by Claude

Lingua principale
TypeScript
Stelle
2
Fork
0
Merge medio
2g 5h
PR unite (30g)
46

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di sillsdev/interlinearizer-extension

Tutte le issue di sillsdev/interlinearizer-extension

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.