fix: mark cohort_definition_id as primary key in cohort_definition table for v5.3 and v5.4

未关闭 适合新手
  • 哪里不对？
#787 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

@rtmalikian 已经在做这个了。

开始于 2026年6月19日。

  • #788 来自 @rtmalikian —— 未关闭

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
82/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
冷清
技术栈
html
领域
databases, documentation

调研方向

先从 inst/csv/OMOP_CDMv5.3_Field_Level.csv 和 inst/csv/OMOP_CDMv5.4_Field_Level.csv 中的 cohort_definition_id 行开始，然后比较 docs/cdm53.html 和 docs/cdm54.html 中对应的条目。确认在两个 CSV 规范和生成的 HTML 文档中，主键值都是 Yes。

由索引模型根据 Issue 内容生成。

描述

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_id has isPrimaryKey=No
  • inst/csv/OMOP_CDMv5.4_Field_Level.csv — same issue
  • docs/cdm53.html — generated HTML shows Primary Key=No
  • docs/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.

主要语言
HTML
星标
1.1k
派生
515
平均合并
8 天 3 小时
30 天内合并 PR
4

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

OHDSI/CommonDataModel 的其他 Issue

查看 OHDSI/CommonDataModel 的全部 Issue

相似的 Issue

更多 Databases Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。