fix: mark cohort_definition_id as primary key in cohort_definition table for v5.3 and v5.4
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- html
- Área
- databases, documentation
Línea de trabajo
Comienza con las filas de cohort_definition_id en inst/csv/OMOP_CDMv5.3_Field_Level.csv y inst/csv/OMOP_CDMv5.4_Field_Level.csv; después, compara sus entradas correspondientes en docs/cdm53.html y docs/cdm54.html. Confirma que el valor de la clave primaria sea Yes en ambas especificaciones CSV y en la documentación HTML generada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The cohort_definition_id field in the cohort_definition table is not marked as a primary key in the CSV spec files for v5.3 and v5.4, even though it is the natural primary key for that table (each cohort definition has a unique ID). The v6.0 CSV correctly marks it as a primary key.
Affected Files
inst/csv/OMOP_CDMv5.3_Field_Level.csv—cohort_definition,cohort_definition_idhasisPrimaryKey=Noinst/csv/OMOP_CDMv5.4_Field_Level.csv— same issuedocs/cdm53.html— generated HTML shows Primary Key=Nodocs/cdm54.html— generated HTML shows Primary Key=No
Proposed Fix
Change isPrimaryKey from No to Yes for the cohort_definition_id field in the cohort_definition table in both v5.3 and v5.4 CSV files, and update the generated HTML docs accordingly.
- Lenguaje dominante
- HTML
- Estrellas
- 1.1k
- Forks
- 515
- Merge medio
- 8 d 3 h
- PR fusionados (30 d)
- 4
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
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OHDSI/CommonDataModel
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
OHDSI/CommonDataModel#802 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
OHDSI/CommonDataModel#799 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
OHDSI/CommonDataModel#784 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
OHDSI/CommonDataModel#823 ·
-
Changes to quarto docsAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
OHDSI/CommonDataModel#797 · 1 comentario ·
Todos los issues de OHDSI/CommonDataModel
Issues similares
-
cloudflare-cache-binding: `unstorage://` keys are rejected by the Cache API, cache never storesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
whatyouhide/redix#358 ·
-
chore severity:low
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Add: Animal Planet BGAbiertochannels:add check:passed
Dificultad 2/5 1-3 horas Aptitud para principiantes 63/100
Los mantenedores suelen responder en 4 días
-
ai-driven-fuzzing bug claude Minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
linagora/lemonldap-ng-plugins#169 ·
Los mantenedores suelen responder en 1 día