Skip to content

Commit

Permalink
Cr uplift merged (#335)
Browse files Browse the repository at this point in the history
* WIP

* further cleanup

* WIP

* WIP

* WIP

* WIP

* WIP

* Holy coly, that's a lot of movement

* WIP

* Pom cleanup

* Remove thunder tests... there's no server here?

* WIP IGRepositoryImplementation

* WIP

* add logger

* remove some commented code

* First pass at search in IG repo

* fix build

* PlanDef tests

* More tests passing

* Fix questionnaire cql location

* Fix parameters

* move tests

* Add search params library name and version

* Fix search by name and version

* cleanup pom

* Add data provider for additional bundle

* Add support for NpmProcessor in Engines

* Disable Cli tests

* Add FederatedDataProvider

* Fix test data

* Delete builder and library

* cleanup

* Fix poms

* Fix loop

* Fix questionnaire benchmark

---------

Co-authored-by: Jonathan Percival <jonathan.i.percival@gmail.com>
  • Loading branch information
barhodes and JPercival committed Aug 31, 2023
1 parent 82db6d6 commit ce49600
Show file tree
Hide file tree
Showing 770 changed files with 64,323 additions and 351,857 deletions.
5 changes: 4 additions & 1 deletion .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
"cdshooks",
"clinicalreasoning",
"Codeable",
"CODEABLECONCEPT",
"codesystems",
"Configurer",
"connectathon",
Expand All @@ -27,6 +28,7 @@
"Fhir",
"fhirpath",
"Gson",
"hamcrest",
"helloworld",
"INITIALPOPULATION",
"INPROGRESS",
Expand All @@ -48,6 +50,7 @@
"opioidcds",
"PATIENTLIST",
"plandefinition",
"Pointcut",
"ppca",
"prepopulate",
"proxying",
Expand Down Expand Up @@ -127,7 +130,7 @@
},
"editor.formatOnSave": true,
"editor.codeActionsOnSave": {
"source.organizeImports": true
// "source.organizeImports": true
},
"editor.detectIndentation": false,
"editor.foldingImportsByDefault": true,
Expand Down
30 changes: 7 additions & 23 deletions cqf-fhir-benchmark/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
</dependency>

<dependency>
<groupId>org.opencds.cqf.cql</groupId>
<artifactId>evaluator.jackson-deps</artifactId>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-jackson</artifactId>
<version>3.0.0-PRE6-SNAPSHOT</version>
<type>pom</type>
</dependency>
Expand All @@ -33,7 +33,11 @@
<artifactId>cqf-fhir-utility</artifactId>
<version>3.0.0-PRE6-SNAPSHOT</version>
</dependency>

<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-test</artifactId>
<version>3.0.0-PRE6-SNAPSHOT</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.cql</groupId>
<artifactId>evaluator.measure-hapi</artifactId>
Expand Down Expand Up @@ -73,7 +77,6 @@
<type>test-jar</type>
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.skyscreamer</groupId>
<artifactId>jsonassert</artifactId>
Expand All @@ -87,30 +90,11 @@
<optional>true</optional>
<scope>test</scope>
</dependency>
<dependency>
<groupId>info.cqframework</groupId>
<artifactId>model-jackson</artifactId>
<version>${cql.version}</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>info.cqframework</groupId>
<artifactId>elm-jackson</artifactId>
<version>${cql.version}</version>
<scope>test</scope>
</dependency>
</dependencies>

<build>
<pluginManagement>
<plugins>
<plugin>
<artifactId>maven-deploy-plugin</artifactId>
<version>3.0.0-M1</version>
<configuration>
<skip>true</skip>
</configuration>
</plugin>
<plugin>
<groupId>org.sonatype.plugins</groupId>
<artifactId>nexus-staging-maven-plugin</artifactId>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,13 +1,14 @@
package org.opencds.cqf.fhir.benchmark;

import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.testng.Assert.assertFalse;
import static org.testng.Assert.assertTrue;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import java.text.DecimalFormat;
import java.util.Collection;
import java.util.Map;

import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
import org.openjdk.jmh.results.RunResult;
import org.openjdk.jmh.runner.Runner;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
package org.opencds.cqf.fhir.benchmark;

import static org.opencds.cqf.cql.evaluator.fhir.util.r4.Parameters.parameters;
import static org.opencds.cqf.cql.evaluator.fhir.util.r4.Parameters.part;
import static org.opencds.cqf.fhir.utility.r4.Parameters.parameters;
import static org.opencds.cqf.fhir.utility.r4.Parameters.part;

import java.util.Collection;
import java.util.concurrent.TimeUnit;

import org.hl7.fhir.r4.model.IdType;
import org.opencds.cqf.cql.evaluator.fhir.test.TestRepositoryFactory;
import org.opencds.cqf.cql.evaluator.plandefinition.r4.PlanDefinition;
import org.opencds.cqf.cql.evaluator.plandefinition.r4.PlanDefinition.Apply;
import org.opencds.cqf.fhir.api.Repository;
import org.opencds.cqf.fhir.test.TestRepositoryFactory;
import org.openjdk.jmh.annotations.Benchmark;
import org.openjdk.jmh.annotations.Fork;
import org.openjdk.jmh.annotations.Level;
Expand All @@ -33,7 +33,7 @@
public class PlanDefinitions {
private static final FhirContext FHIR_CONTEXT = FhirContext.forR4Cached();
private static final Repository REPOSITORY = TestRepositoryFactory.createRepository(FHIR_CONTEXT,
PlanDefinition.Assert.class, "anc-dak");
PlanDefinition.Assert.class, PlanDefinition.CLASS_PATH + "/anc-dak");

private Apply apply;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
package org.opencds.cqf.fhir.benchmark;

import static org.opencds.cqf.cql.evaluator.fhir.util.r4.Parameters.parameters;
import static org.opencds.cqf.cql.evaluator.fhir.util.r4.Parameters.stringPart;
import static org.opencds.cqf.fhir.utility.r4.Parameters.parameters;
import static org.opencds.cqf.fhir.utility.r4.Parameters.stringPart;

import java.util.Collection;
import java.util.concurrent.TimeUnit;

import org.hl7.fhir.r4.model.IdType;
import org.opencds.cqf.cql.evaluator.fhir.test.TestRepositoryFactory;
import org.opencds.cqf.cql.evaluator.questionnaire.r4.QuestionnaireProcessorTests;
import org.opencds.cqf.cql.evaluator.questionnaire.r4.helpers.TestQuestionnaire;
import org.opencds.cqf.cql.evaluator.questionnaire.r4.helpers.TestQuestionnaire.QuestionnaireResult;
import org.opencds.cqf.fhir.api.Repository;
import org.opencds.cqf.fhir.test.TestRepositoryFactory;
import org.openjdk.jmh.annotations.Benchmark;
import org.openjdk.jmh.annotations.Fork;
import org.openjdk.jmh.annotations.Level;
Expand All @@ -33,17 +33,18 @@
public class Questionnaires {
private static final FhirContext FHIR_CONTEXT = FhirContext.forR4Cached();
private static final Repository REPOSITORY = TestRepositoryFactory.createRepository(FHIR_CONTEXT,
QuestionnaireProcessorTests.class, "pa-aslp");
QuestionnaireProcessorTests.class, TestQuestionnaire.CLASS_PATH + "/pa-aslp");

private QuestionnaireResult result;

@Setup(Level.Iteration)
public void setupIteration() throws Exception {
this.result = TestQuestionnaire.Assert.that(new IdType("Questionnaire", "ASLPA1"), "positive")
.withRepository(REPOSITORY)
.withParameters(parameters(stringPart("Service Request Id", "SleepStudy"),
stringPart("Service Request Id", "SleepStudy2"),
stringPart("Coverage Id", "Coverage-positive")));
this.result =
TestQuestionnaire.Assert.that(new IdType("Questionnaire", "ASLPA1"), "positive")
.withRepository(REPOSITORY)
.withParameters(parameters(stringPart("Service Request Id", "SleepStudy"),
stringPart("Service Request Id", "SleepStudy2"),
stringPart("Coverage Id", "Coverage-positive")));
}

@Benchmark
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@

import org.opencds.cqf.cql.engine.runtime.Code;
import org.opencds.cqf.cql.engine.terminology.ValueSetInfo;
import org.opencds.cqf.cql.evaluator.engine.terminology.RepositoryTerminologyProvider;
import org.opencds.cqf.cql.evaluator.fhir.test.TestRepositoryFactory;
import org.opencds.cqf.cql.evaluator.measure.r4.MeasureProcessorEvaluateTest;
import org.opencds.cqf.fhir.cql.engine.terminology.RepositoryTerminologyProvider;
import org.opencds.cqf.fhir.test.TestRepositoryFactory;
import org.openjdk.jmh.annotations.Benchmark;
import org.openjdk.jmh.annotations.Fork;
import org.openjdk.jmh.annotations.Level;
Expand Down
25 changes: 0 additions & 25 deletions cqf-fhir-bom/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -39,26 +39,6 @@
<artifactId>cqf-fhir-cql</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-cql-dstu3</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-cql-r4</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-cql-r5</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-elm</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-jackson</artifactId>
Expand All @@ -76,11 +56,6 @@
<artifactId>cqf-fhir-cr</artifactId>
<version>${project.version}</version>
</dependency>
<dependency>
<groupId>org.opencds.cqf.fhir</groupId>
<artifactId>cqf-fhir-cdshooks</artifactId>
<version>${project.version}</version>
</dependency>
</dependencies>
</dependencyManagement>
</project>
48 changes: 0 additions & 48 deletions cqf-fhir-cdshooks/pom.xml

This file was deleted.

This file was deleted.

27 changes: 0 additions & 27 deletions cqf-fhir-cql-dstu3/pom.xml

This file was deleted.

This file was deleted.

27 changes: 0 additions & 27 deletions cqf-fhir-cql-r4/pom.xml

This file was deleted.

This file was deleted.

27 changes: 0 additions & 27 deletions cqf-fhir-cql-r5/pom.xml

This file was deleted.

Loading

0 comments on commit ce49600

Please sign in to comment.