Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Backport MeasureReport.subject from R5 #4508

Closed
rahul1 opened this issue May 3, 2024 · 1 comment · Fixed by #4589
Closed

Backport MeasureReport.subject from R5 #4508

rahul1 opened this issue May 3, 2024 · 1 comment · Fixed by #4589
Assignees
@codyebberson
Labels
fhir-datastore Related to the FHIR datastore, includes API and FHIR operations
Milestone
May 31st, 2024

Comments

@rahul1
Copy link
Member

rahul1 commented May 3, 2024 •
edited

Use Case

In many VBC use cases, quality measures (e.g. cost score, risk score, quality score) are computed not only on Practitioners, but on the Organization level (e.g. Hospital cost score).

In R4 and R4b, MeasureReport.subject can point to a Patient and Practitioner (amongst other resourceTypes), but not to a Organization. This was rectified in R5

Proposal

  • Backport the choice of types for MeasureReport.subject from R5 -> R4. I acknowledge that this might be controversial, because we (rightly) want to avoid creating a chimera of mixed R4/R5 models - so I'm open to alternatives.

Alternatives Considered

  • Using Observation instead of MeasureReport: This has the same problem MeasureReport - subject can be an Organization in R5, but not R4
  • Setting MeasureReport.subject to a Location : This breaks the semantics of Location - In general an organization (e.g. hospital system) might have 2+ locations (facilities, buildings). So associating the the MeasureReport with a Location doesn't have the right cardinality, unless we enforce making a "dummy" Location
@reshmakh reshmakh added the fhir-datastore Related to the FHIR datastore, includes API and FHIR operations label May 3, 2024
@rahul1 rahul1 added this to the May 31st, 2024 milestone May 7, 2024
@codyebberson codyebberson self-assigned this May 7, 2024
codyebberson added a commit that referenced this issue May 7, 2024
@codyebberson
Fixes #4508 - MeasureReport-subject search param backport
4d94eab
codyebberson added a commit that referenced this issue May 9, 2024
@codyebberson
Fixes #4508 - MeasureReport-subject search param backport
1b41542
github-merge-queue bot pushed a commit that referenced this issue May 9, 2024
@codyebberson
Fixes #4508 - MeasureReport-subject search param backport (#4530)
f610273 
* Fixes #4508 - MeasureReport-subject search param backport

* Updated StructureDefinition
@codyebberson
Copy link
Member

Fixed in #4530

@medplumbot medplumbot mentioned this issue May 25, 2024
Merged
medplumbot added a commit that referenced this issue May 25, 2024
@medplumbot
Release Version 3.1.6
5a493cf 
Fix age display in PatientSummary (#4484)Fixes #4471 - DetectedIssue.status valueset and search param (#4483)
Qualify columns with table name in generated SQL (#4487)
fix(migrations): only take lock if migrating (#4490)
Document updating profiles (#4402)
Run expand tests against old and new (#4503)
Adding OpenCareHub support post (#4504)
[Medplum Provider app] Various fixes and touchups (#4500)
ci(agent): add workflow for building agent outside of a release (#4512)
feat(agent): `Agent/$reload-config` operation (#4457)
Remove broken links to Foo Provider (#4518)
Dependency upgrades 2024-05-06 (#4515)
PatientSummary and provider app tweaks (#4521)
Fixes #4509 - Improve exact match search support for token parameters in `matchesTokenFilter` (#4516)
Link to new Demo Applications (#4514)
Clarify that autobatching only applies to `GET` requests (#4479)
Added missing useEffect dependency in chat demo (#4527)
fix: allows CORS for `keyvalue` API (#4476)
Fixes #4508 - MeasureReport-subject search param backport (#4530)
Fix CLI update-server version flag (#4534)
Patient summary appointments and encounters links (#4524)
Remove unused DB columns (#4532)
fix(agent): unwrap response for `$reload-config` by id (#4542)
`PatientSummary` Problem List uses US Core profile (#4535)
fix(cli): always exit with exit code 1 after error occurs during command (#4536)
Add failing test about validating nested extensions (#4548)
Add error message when `cli` fails on login (#4507)
Remove functions moved to core (#4547)
fix: AttachmentDisplay use uncached url for download link (#4501)
feat(agent): respect `Agent.status` and `Agent.channel.endpoint.status` being `off` (#4523)
CMS 1500 and Superbill (#4543)
Demo Bot: Agent Setup (#4555)
feat(Subscription): add `author` as a `SearchParameter` (#4540)
Dependency upgrades 2024-05-13 (#4544)
Full linked Project ordering in CodeSystem lookup (#4522)
Disable super admin refresh tokens (#4492)
Minor fixes for the agent setup bot (#4560)
docs(agent): document how logging works with `Bot` and `Agent` (#4563)
Split rate limits into two buckets (#4568)
Properly detect array elements (#4569)
Apply filter to ValueSet with expansion.contains (#4570)
More efficiently validate included concepts (#4573)
Dependency upgrades 2024-05-20 (#4574)
tweak(agent): add timezone in status `lastUpdated` time (#4564)
fix(client/keyvalue): set keyvalue content-type text (#4575)
Allow configuring server default rate limits (#4491)
feat(cli): add `token` command to get access token (#4579)
Updating device resources and videos (#4578)
fix(subscriptions): don't retry ws subs if sub is deleted (#4577)
Add support for 'pr' filter operation (#4584)
Super admin endpoint for database stats (#4443)
github-merge-queue bot pushed a commit that referenced this issue May 25, 2024
@medplumbot
Release Version 3.1.6 (#4589)
177ea04 
Fix age display in PatientSummary (#4484)Fixes #4471 - DetectedIssue.status valueset and search param (#4483)
Qualify columns with table name in generated SQL (#4487)
fix(migrations): only take lock if migrating (#4490)
Document updating profiles (#4402)
Run expand tests against old and new (#4503)
Adding OpenCareHub support post (#4504)
[Medplum Provider app] Various fixes and touchups (#4500)
ci(agent): add workflow for building agent outside of a release (#4512)
feat(agent): `Agent/$reload-config` operation (#4457)
Remove broken links to Foo Provider (#4518)
Dependency upgrades 2024-05-06 (#4515)
PatientSummary and provider app tweaks (#4521)
Fixes #4509 - Improve exact match search support for token parameters in `matchesTokenFilter` (#4516)
Link to new Demo Applications (#4514)
Clarify that autobatching only applies to `GET` requests (#4479)
Added missing useEffect dependency in chat demo (#4527)
fix: allows CORS for `keyvalue` API (#4476)
Fixes #4508 - MeasureReport-subject search param backport (#4530)
Fix CLI update-server version flag (#4534)
Patient summary appointments and encounters links (#4524)
Remove unused DB columns (#4532)
fix(agent): unwrap response for `$reload-config` by id (#4542)
`PatientSummary` Problem List uses US Core profile (#4535)
fix(cli): always exit with exit code 1 after error occurs during command (#4536)
Add failing test about validating nested extensions (#4548)
Add error message when `cli` fails on login (#4507)
Remove functions moved to core (#4547)
fix: AttachmentDisplay use uncached url for download link (#4501)
feat(agent): respect `Agent.status` and `Agent.channel.endpoint.status` being `off` (#4523)
CMS 1500 and Superbill (#4543)
Demo Bot: Agent Setup (#4555)
feat(Subscription): add `author` as a `SearchParameter` (#4540)
Dependency upgrades 2024-05-13 (#4544)
Full linked Project ordering in CodeSystem lookup (#4522)
Disable super admin refresh tokens (#4492)
Minor fixes for the agent setup bot (#4560)
docs(agent): document how logging works with `Bot` and `Agent` (#4563)
Split rate limits into two buckets (#4568)
Properly detect array elements (#4569)
Apply filter to ValueSet with expansion.contains (#4570)
More efficiently validate included concepts (#4573)
Dependency upgrades 2024-05-20 (#4574)
tweak(agent): add timezone in status `lastUpdated` time (#4564)
fix(client/keyvalue): set keyvalue content-type text (#4575)
Allow configuring server default rate limits (#4491)
feat(cli): add `token` command to get access token (#4579)
Updating device resources and videos (#4578)
fix(subscriptions): don't retry ws subs if sub is deleted (#4577)
Add support for 'pr' filter operation (#4584)
Super admin endpoint for database stats (#4443)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codyebberson codyebberson

Labels
fhir-datastore Related to the FHIR datastore, includes API and FHIR operations
Projects
Medplum Product Roadmap
Status: ✅ Done
Milestone
May 31st, 2024
Development

Successfully merging a pull request may close this issue.

Release Version 3.1.6 medplum/medplum
3 participants
@codyebberson @rahul1 @reshmakh