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

Abierto
  • ¿Algo no está bien?
#309 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
35/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
typescript
Área
backend-api-design, documentation

Línea de trabajo

Comienza con src/types/interlinearizer.d.ts:970 y los usos de importación de PT9 en analysisMerger.ts:339 y :406, y luego revisa la discusión vinculada sobre el significado de confidence en #254. Determina y documenta un significado para confidence, define qué sucede con el otro significado y actualiza la documentación del tipo para que coincida con los campos de PT9 enviados; contempla la migración de datos persistidos si cambia el significado del campo.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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

Lenguaje dominante
TypeScript
Estrellas
2
Forks
0
Merge medio
1 d 8 h
PR fusionados (30 d)
85

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de sillsdev/interlinearizer-extension

Todos los issues de sillsdev/interlinearizer-extension

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.