Repository navigation
Human/mouse marker-validity pilot for Dr. Diehl (part 9/9) - #23
Draft
s28ale-maker wants to merge 3 commits into
Draft
s28ale-maker wants to merge 3 commits into
s28ale-maker wants to merge 3 commits into
Conversation
…. Diehl request) Extends reports/pro_marker_species_support.tsv in place; the 10 legacy columns and 77 legacy rows are unchanged (checked by script). - New columns: species_scope (human/mouse/both/unresolved), human_citation, human_quote, mouse_citation, mouse_quote, note, checked_on. Scope: the 87 directly asserted pairs on mapped CL terms; the 10 "(human)"-labelled pairs appended with species_scope = human, "species from PRO label". - Pilot: CL:0000623, CL:0000980, CL:0000938 (10 pairs). Every quote checked exactly against Europe PMC full text. 1 human (CD56-high: mouse NK cells are not classified by CD56), 9 unresolved (mostly no mouse evidence; CD138 contradicted in mouse and conflicting in human; HLA-DRA not clearly supported in human). - The 5 legacy human rows whose quotes did not verify: corrected in the new columns only (4 verified replacements; plasmablast CD138 is in the pilot). - pro-marker-species-support SKILL.md and reports/README describe the new schema; pro_species_support.py docstring points at curated_mappings.tsv. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…for Dr. Diehl - pro_marker_species_support.tsv: CL:0000938 (a human-specific CL term) rows note "CL term is human-specific; mouse not applicable"; CD56-high and lacks-CD16 become species_scope = human (human side verified); lacks-DRA and CD84 stay unresolved. Rule stated in the skill. - Skill and reports README: the new columns take precedence over the legacy species_support value (legacy value on plasmablast CD138 left as is). - cl_term_issues.md: unfiled candidates for plasmablast "lacks CD138" and CD56-bright NK "lacks HLA-DRA", citing the verified quotes. - docs/planning/2026-10-02_diehl_summary.md: one-page pilot summary with questions for Dr. Diehl. Remaining scope is 67 pairs on 30 CL terms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Sep 29, 2026
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
1 participant
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.
Summary
Commits
0debe25..c4a9683:docs/planning/2026-10-02_diehl_summary.md), trimmed to the three questions she asked.Test plan
Checked in isolation with the same steps as CI (
uv sync --frozen --extra dev):uv run pytest: 599 passed, 96.5% coverageuv run ruff check .,ruff format --check ., anduv run mypy src/soulcap_cl_mappingpass🤖 Generated with Claude Code