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.

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

Need updates of the CQF libs #2172

Open
delcroip opened this issue Sep 15, 2023 · 8 comments · Fixed by #2241
Open

Need updates of the CQF libs #2172

delcroip opened this issue Sep 15, 2023 · 8 comments · Fixed by #2241
Assignees

Comments

@delcroip
Copy link
Collaborator

delcroip commented Sep 15, 2023

Describe the bug
ActivityDefinition/$Apply is not working well (via planDefinition/$apply)

  • resources cannot be managed by DynamicValue
  • CQL not supported in DynamicValue

Component
Evaluator

To Reproduce

Expected behavior
should be able to add contained and inputs on Task/Medication Request

Additional context
Major updates were made on the CQF library, those upgrade were tested successfully on the CQF-Ruler Docker snapshot alphora/cqf-ruler:0.14.2-SNAPSHOT (it creates a medication request )

@omarismail94
Copy link
Contributor

This change will not be as simple as changing the versions of the cqf libraries in the Dependencies file. The engine and evaluator belong to the org.opencds.cqf.cql group Id. However, the group id has been deprecated and now all the Cql libraries reside under the info.cqframework group. Coupled with the API changes, going from 2.x to 3.x will take time

@delcroip
Copy link
Collaborator Author

thanks for the update

@fredhersch
Copy link
Collaborator

@jingtang10 @brynrhodes Can we define the work for @vitorpamplona to complete to resolve this issue here.

@jingtang10
Copy link
Collaborator

@brynrhodes: there's a scheduled engine (evaluator?) release today for 3.00 and translator release 3.2.0. Luke tested on a prerelease version.

but the most important change is the name space change. you should only need a repository implementation now. and now the other providers. does mean the other content needs to be provided by the same impl. this should use both engine and knowledge manager in the android fhir sdk.

@jingtang10
Copy link
Collaborator

luke has successfullly run the example on the image (bryn provided) using the prerelease 7.

@jingtang10
Copy link
Collaborator

the proxy repository should actually be able to query either the content repostiory for knowledge artifacts or the data repository for patient data. so this actually maps to fhir engine and knowledge manager quite well. but it might also fall-through, which is what we want.

@jingtang10
Copy link
Collaborator

@lukeaduncan
Copy link

Here is a bundle of files that I was able to get working. From https://github.com/WorldHealthOrganization/smart-immunizations/tree/cleanup
measlesApplyTest.json.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New
Development

Successfully merging a pull request may close this issue.

6 participants