Skip to content

Populate and evaluate enhancements#655

Merged
barhodes merged 13 commits into
masterfrom
populate-fhirpath-issues2
Apr 21, 2025
Merged

Populate and evaluate enhancements#655
barhodes merged 13 commits into
masterfrom
populate-fhirpath-issues2

Conversation

@barhodes
Copy link
Copy Markdown
Contributor

Refactor $populate operation to add support for FHIRPath variables defined in the SDC IG.

Add support for passing raw CQL parameters to evaluation calls. This allows for parameters of types that don't meet the FHIR Parameters profile.

Add support for passing multiple Library references to evaluation calls and qualifying the library by name within the expression.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 15, 2025

Formatting check succeeded!

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 15, 2025

Codecov Report

Attention: Patch coverage is 88.49206% with 87 lines in your changes missing coverage. Please review.

Project coverage is 71.71%. Comparing base (71a2c08) to head (7e4d3b4).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...cds/cqf/fhir/cr/common/InputParameterResolver.java 84.35% 10 Missing and 13 partials ⚠️
.../fhir/cr/questionnaire/generate/ItemGenerator.java 79.41% 3 Missing and 4 partials ⚠️
...questionnaire/populate/ProcessItemWithContext.java 58.82% 3 Missing and 4 partials ⚠️
...hir/cr/questionnaire/populate/PopulateRequest.java 87.50% 3 Missing and 2 partials ⚠️
...s/cqf/fhir/cr/visitor/DataRequirementsVisitor.java 86.11% 4 Missing and 1 partial ⚠️
...ter/dstu3/ParametersParameterComponentAdapter.java 16.66% 5 Missing ⚠️
...ncds/cqf/fhir/cr/common/IQuestionnaireRequest.java 60.00% 2 Missing and 2 partials ⚠️
...qf/fhir/cr/questionnaire/populate/ProcessItem.java 73.33% 2 Missing and 2 partials ⚠️
...va/org/opencds/cqf/fhir/utility/CqfExpression.java 88.57% 2 Missing and 2 partials ⚠️
...n/java/org/opencds/cqf/fhir/cql/LibraryEngine.java 88.46% 2 Missing and 1 partial ⚠️
... and 12 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #655      +/-   ##
============================================
+ Coverage     70.95%   71.71%   +0.75%     
  Complexity      153      153              
============================================
  Files           473      473              
  Lines         22189    22176      -13     
  Branches       2898     2848      -50     
============================================
+ Hits          15744    15903     +159     
+ Misses         4990     4857     -133     
+ Partials       1455     1416      -39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@barhodes barhodes marked this pull request as ready for review April 18, 2025 14:58
@sonarqubecloud
Copy link
Copy Markdown

@barhodes barhodes merged commit db89bc6 into master Apr 21, 2025
10 checks passed
@barhodes barhodes deleted the populate-fhirpath-issues2 branch April 21, 2025 19:34
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.

2 participants