fix: mark cohort_definition_id as primary key in cohort_definition table for v5.3 and v5.4
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 82/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- html
- Lĩnh vực
- databases, documentation
Hướng nghiên cứu
Bắt đầu với các hàng cohort_definition_id trong inst/csv/OMOP_CDMv5.3_Field_Level.csv và inst/csv/OMOP_CDMv5.4_Field_Level.csv, sau đó so sánh các mục tương ứng trong docs/cdm53.html và docs/cdm54.html. Xác nhận rằng giá trị khóa chính là Yes trong cả hai đặc tả CSV và tài liệu HTML được tạo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- HTML
- Star
- 1.1k
- Fork
- 515
- Merge trung bình
- 8 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 4
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của OHDSI/CommonDataModel
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 74/100
OHDSI/CommonDataModel#802 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
OHDSI/CommonDataModel#799 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
OHDSI/CommonDataModel#784 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
OHDSI/CommonDataModel#823 ·
-
Changes to quarto docsĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
OHDSI/CommonDataModel#797 · 1 bình luận ·
Tất cả issue của OHDSI/CommonDataModel
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
oracle/rust-oracledb#66 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
drizzle-team/drizzle-orm#6639 ·
Maintainer thường phản hồi trong vòng 4 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apache/grails-core#16874 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ahmedmmordi/store-app#8 ·
-
series passes: Stripe activation backfills tickets without rebuilding attendee visibilityCó thể đã có người làm @biagiodistefano đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
letsrevel/revel-backend#1309 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày