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

Updating the PlanDefinition PR based on reviewer's request #74

Merged
merged 96 commits into from
Dec 17, 2021
Merged

Updating the PlanDefinition PR based on reviewer's request #74

merged 96 commits into from
Dec 17, 2021

Conversation

vitorpamplona
Copy link
Contributor

@vitorpamplona vitorpamplona commented Dec 17, 2021 •
edited
Loading

  1. Bringing the Branch PlanDefinitionModule up to speed with Master.
  2. Solving many of the simpler issues raised by @jp on that Review.
  3. Simplifying the code a bit. There are significant improvements that could be made to improve code readability.
  4. A more complete Test suite is welcomed.

csandersdev and others added 30 commits June 10, 2021 16:14
@csandersdev
cqframework#43 - Remove HAPI dependency from evaluator.measure
5a4746d
@csandersdev
Fix javadoc error caused by mismatched package paths
2e2d568
@csandersdev
Merge branch 'master' of github.com:DBCG/cql-evaluator
63b3dc6
@JPercival
Merge pull request cqframework#44 from csandersdev/master
e64d248 
cqframework#43 - Remove HAPI dependency from evaluator.measure
@csandersdev
Implement supplemental data
351db5e 
I brought over the supplemental data logic from cqf-ruler and made it
pluggable for different FHIR versions / models.
@csandersdev
Merge branch 'master' of github.com:DBCG/cql-evaluator
d68368c
@JPercival
Merge pull request cqframework#48 from csandersdev/master
9010d62 
Implement support for supplemental data
@JPercival
Measure processor for HAPI
a212539
@JPercival
Make evaluateMeasure public
35b3871
@JPercival
Fix incorrect Url type
a998668
@JPercival
Fixes for Measure eval
415c84d
@JPercival
Added test for qualified Patient references
2593397
@JPercival
Refactoring to make measure eval more self-contained
7fec024
@JPercival
Merge pull request cqframework#49 from DBCG/feature-measure-eval-refa…
b788a3a 
…ctoring

Refactoring to make measure eval more self-contained
@JPercival
Initial stratification test
ff482e6
@JPercival
Stratification implementation
a676386
@JPercival
Addtional tests / bug fixes
dfa6271
@JPercival
Bug fixes
ac90638
@JPercival
Score tests and bug fixes
e0fe612
@JPercival
More tests and bug fixes
cf7d193
@JPercival
Fix errorneous class reference
e06045c
@JPercival
More tests, some documentation, initial DSTU3 implementation
fe052d5
@JPercival
Spring test for dstu3
b262d87
@JPercival
Initial support for local server data
5c382cc
@JPercival
Merge pull request cqframework#50 from DBCG/feature-stratification
7361532 
Feature stratification
@JPercival
Clean up of MeasureReports
9770739
@JPercival
Dedup extensions
a7e29f4
@JPercival
More fixes for stratum text
6a94cd1
@JPercival
More tests and bug fixes for Measure eval
13495db
@JPercival
More fixes for stratification, additional tests
a8abc44
jreyno77 and others added 25 commits October 19, 2021 09:45
@jreyno77
Added a test for checking SDE reference in evaluatedResources
cb1dd8f 
removed extension from SDE
@JPercival
Merge pull request cqframework#66 from DBCG/FixSDEExtension
9ce6578 
Added a test for checking SDE reference in evaluatedResources
@mdnazmulkarim
Add stratifier fixture
d98101f
@mdnazmulkarim
Update fixture
e311654
@mdnazmulkarim
Update Aggregate fixture
c4e5094
@mdnazmulkarim
Add stratifier merge
a0dbbd3
@mdnazmulkarim
Add extension merge
3801581
@mdnazmulkarim
change small statement flow
7194510
@JPercival
Merge pull request cqframework#67 from DBCG/feature-aggregation
4462bf7 
Feature aggregation
@JPercival
Workaround for Date type Measurement Period parameters
f48b124
@mdnazmulkarim
update hapi fhir 5.5.3
d2054b4
@JPercival
Merge pull request cqframework#72 from DBCG/update-hapi-fhir-version
5514f2b 
Update hapi fhir 5.5.3
@vitorpamplona
Merge remote-tracking branch 'upstream/master' into PlanDefinitionModule
39a2734 
# Conflicts:
#	evaluator.builder/src/main/java/org/opencds/cqf/cql/evaluator/builder/FhirDalFactory.java
#	evaluator.builder/src/main/java/org/opencds/cqf/cql/evaluator/builder/dal/FhirDalFactory.java
#	pom.xml
@vitorpamplona
Updating version numbers to match master
cd7a486
@vitorpamplona
Adding a constructor param due the merge with the master branch
36a0848
@vitorpamplona
Removing print out
3c21ba8
@vitorpamplona
Removing VSCode files
7d9331f
@vitorpamplona
Removing unnecessary Sysout
56a0a98
@vitorpamplona
Removing unnecessary imports
06430b4
@vitorpamplona
Removing unnecessary JavaDoc
00e1095
@vitorpamplona
Removing unnecessary Sysout
97a3413
@vitorpamplona
Keeping the previous version of the engine while the JAXB PR is not a…
d719193 
…pproved.
@vitorpamplona
This core.version is not used anywhere.
3aa195a
@vitorpamplona
Returning a CarePlan on Apply
e7c4446
@vitorpamplona
Simplifying the code
d27f48d
@vitorpamplona vitorpamplona changed the title Updating the PR based on reviewer's request Updating the PlanDefinition PR based on reviewer's request Dec 17, 2021
@JPercival JPercival mentioned this pull request Dec 17, 2021
Merged
@JPercival JPercival merged commit a8f26b1 into cqframework:PlanDefinitionModule Dec 17, 2021
@vitorpamplona vitorpamplona mentioned this pull request Sep 19, 2022
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JPercival JPercival JPercival approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@vitorpamplona @JPercival @csandersdev @jreyno77 @brynrhodes @mdnazmulkarim