Skip to content

provenance: the round trip maps a document label to the stored code - #1572

Open
abrichr wants to merge 1 commit into
mainfrom
fix/roundtrip-value-map
Open

abrichr wants to merge 1 commit into
mainfrom
fix/roundtrip-value-map

Conversation

@abrichr

@abrichr abrichr commented Oct 25, 2026

Copy link
Copy Markdown
Member

A referral prints a state's name (Massachusetts) or a provider's name; the record stores the code (MA, a user id). The round trip compared the stored code with the document's text, so a mapped field could never be round-tripped. A misread provider was saved and verified.

The round trip now applies the field's declared value map ({param: state_label, map: {Massachusetts: MA}}, bound_maps) to the document's value before comparing. A document value the map doesn't name can't be compared, so it's not_checked, not held.

Tests: test_provenance.py::test_round_trip_maps_a_label_to_the_stored_code (MA held, RI refuted).

Draft until the live OpenEMR Phase 0 run.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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