BIOP02-74 docs: split_policy §10 PAM50 소스 문구를 실사용에 맞게 정정 (kkkim 결정 이행) - #148
Merged
Merged
Conversation
kkkim Leader 결정(2026-08-20, Jira 11999)의 종결 조건 3건 중 미이행 상태로 남아 있던 "§10 문구를 실사용에 맞게 정정"을 이행한다. PR #139는 Methods draft만 고쳤고 정책 문서는 손대지 않아, main의 §10이 여전히 "PAM50 소스 = cBioPortal 1순위"로 남아 실사용(Parker 계산본)과 어긋나 있었다. ## split_policy_v0.md - §10: 06-10 결정문을 취소선으로 남기고 대체됐음을 명시. 기록을 지우지 않는다. - §10.1 신설: 정본 소스 = tcga_brca_pam50_computed.csv(Parker 2009 nearest-centroid 계산본), 06-10 문구가 어긋났던 이유(fallback 발동 조건인 "커버리지 부족"이 충족된 적 없음 — 실측 97.2%), kkkim 결정과 근거, 57.0% 불일치가 오류가 아니라 파생 차이라는 점, 비차단 잔여 gap. - §4 PAM50 행·§7 라벨 단일소스 체크항목: §10이 "→ §4·§7에 반영"이라 지시한 두 곳도 같은 부정확성(cBioPortal 유래 서술)을 갖고 있어 함께 정합화. ## paper_a_methods_data_draft.md - "Open item — needs a data-owner decision" 절을 결정 반영본으로 교체. kkkim이 08-20에 이미 결정했는데 문서는 결정 대기로 남아 있었다. - 하단 "미해결 항목(투고 전 kkkim 결정 필요)" 문단도 해소 서술로 교체. - 커버리지 표기 정정: 981/1,010(97.1%) → 981/1,009(97.2%). 커밋된 pam50_source_reconcile_biop02-74.json의 cbio_coverage_of_manifest_cohort_pct는 분모가 manifest PAM50 보유 1,009명인데 초안이 전체 1,010명으로 적어 인용 대상 산출물과 분모가 어긋나 있었다. 정책 변경 아님 — §9 서명 2건과 split_hash 5995f29d3978b831은 손대지 않았고 라벨 값·manifest·split 배정도 불변이다. 문서를 실사용에 맞추는 정합 작업이다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Collaborator
|
kkkim 리뷰 — 정정 후 머지 (블로커 1건) 결정 취지(11999) 반영은 정확합니다(Parker 정본 유지·취소선 보존·투명성 보고·gap 비블로커). 다만 57% 라벨이 역전돼 있어 머지 전 정정이 필요합니다. 실측 아래 두 국문 문장이 이를 "불일치"로 뒤집어 씁니다(영문 L65 "match at 57.0%"는 이미 맞아 자기모순):
→ "57.0% 일치(concordance, 514/902), 43.0% 불일치(388/902)"로 정정하고 혼동쌍 설명은 43% 불일치 집합에 붙여주세요. 안 고치면 "57% 불일치"가 원고로 전파됩니다. 비블로커: 커버리지 분모 통일(§10.1 981/1009=97.2% vs Methods 981/1010=97.1%). 상세 = JIRA BIOP02-74 코멘트. 최종 머지는 cross-review상 @braveji18. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
kkkim Leader 결정(2026-08-20, Jira 11999)의 종결 조건 3건 중 미이행으로 남아 있던 1건을 이행합니다.
PR #139는 Methods draft만 고치고 정책 문서는 건드리지 않아,
main의 §10이 여전히"PAM50 소스 = cBioPortal 1순위"로 남아 실사용(Parker 계산본)과 어긋나 있었습니다.
kkkim님이 리뷰에서 티켓 종결 조건으로 명시한 바로 그 항목입니다.
결정 조건 이행 현황
split_policy_v0.md§10 문구를 실사용에 맞게 정정변경 내용
agents/data/split_policy_v0.mdtcga_brca_pam50_computed.csv(Parker 2009 nearest-centroid 계산본,DOI
10.1200/JCO.2008.18.1370). 근거 =tcga_brca_pam50_computed_PROVENANCE.md(BIOP02-49 후속, kkkim 07-10) — manifest pam50 보유 case 1009/1009(100%) 일치로 소스 확정.
실측 커버리지가 부족하지 않았음(97.2%) → 발동 조건이 충족된 적 없는데 fallback이
전 코호트에 쓰이고 있었음.
지시한 두 곳도 같은 부정확성(cBioPortal 유래 서술)을 갖고 있었습니다. §10만 고치면
문서 안에서 서로 어긋난 채 남아 함께 정합화했습니다.
guide/paper_a_methods_data_draft.md묻고 있었습니다(이미 결정이 난 뒤였음) → 결정 반영본으로 교체.
작업 중 발견한 수치 오류 1건
초안이 커버리지를 **981/1,010 = 97.1%**로 적었는데, 인용 대상인
pam50_source_reconcile_biop02-74.json의cbio_coverage_of_manifest_cohort_pct = 97.2는분모가 1,009(manifest PAM50 보유 환자)입니다(
pam50_source_reconcile.py:35).산술은 양쪽 다 맞지만 인용하는 산출물과 분모가 어긋나 있어 **981/1,009 = 97.2%**로
통일했습니다.
split_hash=5995f29d3978b831불변문서를 실사용에 맞추는 정합 작업이며, 수치·데이터는 재계산하지 않았습니다.
리뷰 요청
split_policy_v0.md은 jamie 작성물이라 Cross-review 규칙상 @braveji 님 리뷰 대상입니다.sign-off lock 문서라 셀프 머지는 피하는 게 맞다고 판단해 머지도 부탁드립니다.
결정 당사자이신 @kkkim 님도 §10.1이 11999 결정 취지를 정확히 옮겼는지 확인 부탁드립니다.
두 가지 참고: