Skip to content

OpenConceptLab/ocl_issues#2903 | Add the Mapper's sample spreadsheet, so it can change without a Mapper release - #71

Merged
paynejd merged 1 commit into
mainfrom
docs/2903-mapper-sample
Oct 15, 2026
Merged

paynejd merged 1 commit into
mainfrom
docs/2903-mapper-sample

Conversation

@paynejd

@paynejd paynejd commented Oct 15, 2026

Copy link
Copy Markdown
Member

Part of OpenConceptLab/ocl_issues#2903.

Adds the sample spreadsheet that the Mapper's empty project list offers to new users (OpenConceptLab/oclmap#103), so it can change without a Mapper release.

  • The Mapper downloads it from https://raw.githubusercontent.com/OpenConceptLab/ocl-docs/main/docs/source/oclmap/samples/mapper_sample_lab_tests.csv. An edit merged to main reaches users within about five minutes, GitHub's cache time.
  • If that download fails (a network that blocks raw.githubusercontent.com, or the file has moved), the Mapper falls back to the copy it bundles. Keep this path stable: if the file moves or is renamed, users quietly get the bundled copy instead.
  • The content is the same 10 lab tests as oclmap's public/mapper_sample_lab_tests.csv, with code, name and specimen columns.
  • Sphinx builds only .rst and .md files, so this file doesn't change the docs build. The Mapper docs can link it later.

Merge order doesn't matter: until this is on main, the Mapper uses its bundled copy.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AmWWeGES7vePPDgaxbWusF

… so it can change without a Mapper release

The Mapper's empty project list (OpenConceptLab/oclmap#103) downloads this
file from raw.githubusercontent.com, falling back to the copy bundled with
the Mapper. An edit here reaches users within about five minutes, GitHub's
cache time. Keep the path stable: if the file moves or is renamed, users
get the bundled copy instead.

Same 10 lab tests as oclmap's public/mapper_sample_lab_tests.csv.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AmWWeGES7vePPDgaxbWusF
@paynejd

paynejd commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

Codex review, pass 1 (codex-cli 0.162.1, commit 781aa5c)

Verdict: clean. No findings.

findings.json
{
  "reviewer": "codex-cli 0.162.1",
  "perspective": "defects",
  "mode": "full",
  "commit": "781aa5c38140482563d1007d94a9bba26a4653aa",
  "verdict": "clean",
  "prior": [],
  "findings": []
}

Simplification review (Claude subagent, commit 781aa5c)

Verdict: clean. Nothing to cut.

simplify.json
{
  "reviewer": "claude subagent",
  "perspective": "simplify",
  "mode": "full",
  "commit": "781aa5c38140482563d1007d94a9bba26a4653aa",
  "verdict": "clean",
  "prior": [],
  "findings": []
}

Also checked: the file is byte-identical to oclmap's public/mapper_sample_lab_tests.csv. Tested in the Mapper with the change proposed on OpenConceptLab/oclmap#103: until this merges, the GitHub request returns 404 and the Mapper downloads its bundled copy; with the file served from GitHub, the GitHub copy downloads.

@paynejd
paynejd merged commit 84233b7 into main Oct 15, 2026
@paynejd
paynejd deleted the docs/2903-mapper-sample branch October 15, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant