diff options
author | Harald Musum <musum@yahooinc.com> | 2022-02-11 07:40:33 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-02-11 07:40:33 +0100 |
commit | 9d09b3c253afb08d679d7e3d448c479a29bc58c2 (patch) | |
tree | 8c79aefa0869cdf9ecb07f82f57d5bb7dee6b039 | |
parent | 4213e2ce69f07e9719958c280d331c5e28eaa568 (diff) |
Move method for checking if onnxruntime is available
Use it in model-evaluation as well
5 files changed, 25 insertions, 23 deletions
diff --git a/model-evaluation/src/test/java/ai/vespa/models/evaluation/OnnxEvaluatorTest.java b/model-evaluation/src/test/java/ai/vespa/models/evaluation/OnnxEvaluatorTest.java index c5559d9bed5..27d1c08ea39 100644 --- a/model-evaluation/src/test/java/ai/vespa/models/evaluation/OnnxEvaluatorTest.java +++ b/model-evaluation/src/test/java/ai/vespa/models/evaluation/OnnxEvaluatorTest.java @@ -1,11 +1,10 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package ai.vespa.models.evaluation; +import ai.vespa.modelintegration.evaluator.OnnxEvaluator; import com.yahoo.config.subscription.ConfigGetter; -import com.yahoo.config.subscription.FileSource; import com.yahoo.filedistribution.fileacquirer.FileAcquirer; import com.yahoo.filedistribution.fileacquirer.MockFileAcquirer; -import com.yahoo.path.Path; import com.yahoo.tensor.Tensor; import com.yahoo.vespa.config.search.RankProfilesConfig; import com.yahoo.vespa.config.search.core.OnnxModelsConfig; @@ -19,6 +18,7 @@ import java.util.Map; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; +import static org.junit.Assume.assumeTrue; /** * @author lesters @@ -30,6 +30,7 @@ public class OnnxEvaluatorTest { @Test public void testOnnxEvaluation() { + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); ModelsEvaluator models = createModels(); assertTrue(models.models().containsKey("add_mul")); diff --git a/model-evaluation/src/test/java/ai/vespa/models/handler/ModelsEvaluationHandlerTest.java b/model-evaluation/src/test/java/ai/vespa/models/handler/ModelsEvaluationHandlerTest.java index bb442d76763..215e230b45d 100644 --- a/model-evaluation/src/test/java/ai/vespa/models/handler/ModelsEvaluationHandlerTest.java +++ b/model-evaluation/src/test/java/ai/vespa/models/handler/ModelsEvaluationHandlerTest.java @@ -1,6 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package ai.vespa.models.handler; +import ai.vespa.modelintegration.evaluator.OnnxEvaluator; import ai.vespa.models.evaluation.ModelsEvaluator; import ai.vespa.models.evaluation.RankProfilesConfigImporterWithMockedConstants; import com.yahoo.config.subscription.ConfigGetter; @@ -18,6 +19,8 @@ import org.junit.Test; import java.util.HashMap; import java.util.Map; +import static org.junit.Assume.assumeTrue; + public class ModelsEvaluationHandlerTest { private static final String MODELS_DIR = "src/test/resources/config/models/"; @@ -244,6 +247,7 @@ public class ModelsEvaluationHandlerTest { @Test public void testMnistSavedEvaluateSpecificFunction() { + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); Map<String, String> properties = new HashMap<>(); properties.put("input", inputTensor()); String url = "http://localhost/model-evaluation/v1/mnist_saved/serving_default.y/eval"; diff --git a/model-evaluation/src/test/java/ai/vespa/models/handler/OnnxEvaluationHandlerTest.java b/model-evaluation/src/test/java/ai/vespa/models/handler/OnnxEvaluationHandlerTest.java index ccd303990c8..f065435ec15 100644 --- a/model-evaluation/src/test/java/ai/vespa/models/handler/OnnxEvaluationHandlerTest.java +++ b/model-evaluation/src/test/java/ai/vespa/models/handler/OnnxEvaluationHandlerTest.java @@ -1,12 +1,11 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package ai.vespa.models.handler; +import ai.vespa.modelintegration.evaluator.OnnxEvaluator; import ai.vespa.models.evaluation.ModelsEvaluator; import com.yahoo.config.subscription.ConfigGetter; -import com.yahoo.config.subscription.FileSource; import com.yahoo.filedistribution.fileacquirer.FileAcquirer; import com.yahoo.filedistribution.fileacquirer.MockFileAcquirer; -import com.yahoo.path.Path; import com.yahoo.tensor.Tensor; import com.yahoo.vespa.config.search.RankProfilesConfig; import com.yahoo.vespa.config.search.core.OnnxModelsConfig; @@ -19,6 +18,8 @@ import java.io.File; import java.util.HashMap; import java.util.Map; +import static org.junit.Assume.assumeTrue; + public class OnnxEvaluationHandlerTest { private static HandlerTester handler; @@ -26,6 +27,7 @@ public class OnnxEvaluationHandlerTest { @BeforeClass static public void setUp() { + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); handler = new HandlerTester(createModels()); } diff --git a/model-integration/src/main/java/ai/vespa/modelintegration/evaluator/OnnxEvaluator.java b/model-integration/src/main/java/ai/vespa/modelintegration/evaluator/OnnxEvaluator.java index 87b964a2c56..c9ab9924214 100644 --- a/model-integration/src/main/java/ai/vespa/modelintegration/evaluator/OnnxEvaluator.java +++ b/model-integration/src/main/java/ai/vespa/modelintegration/evaluator/OnnxEvaluator.java @@ -93,4 +93,13 @@ public class OnnxEvaluator { } } + public static boolean isRuntimeAvailable() { + try { + new OnnxEvaluator(""); + return true; + } catch (UnsatisfiedLinkError | RuntimeException | NoClassDefFoundError e) { + return false; + } + } + } diff --git a/model-integration/src/test/java/ai/vespa/modelintegration/evaluator/OnnxEvaluatorTest.java b/model-integration/src/test/java/ai/vespa/modelintegration/evaluator/OnnxEvaluatorTest.java index bfb441443fc..6266dcef174 100644 --- a/model-integration/src/test/java/ai/vespa/modelintegration/evaluator/OnnxEvaluatorTest.java +++ b/model-integration/src/test/java/ai/vespa/modelintegration/evaluator/OnnxEvaluatorTest.java @@ -16,15 +16,11 @@ import static org.junit.Assume.assumeTrue; * @author lesters */ public class OnnxEvaluatorTest { - private static final String simpleModelPath = "src/test/models/onnx/simple/simple.onnx"; - // Check if onnxruntime is available, needs to be done only once due to static instance - // variable in OrtEnvironment - private static final boolean onnxRuntimeIsAvailable = onnxRuntimeIsAvailable(); @Test public void testSimpleModel() { - assumeTrue(onnxRuntimeIsAvailable); - OnnxEvaluator evaluator = new OnnxEvaluator(simpleModelPath); + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); + OnnxEvaluator evaluator = new OnnxEvaluator("src/test/models/onnx/simple/simple.onnx"); // Input types Map<String, TensorType> inputTypes = evaluator.getInputInfo(); @@ -48,7 +44,7 @@ public class OnnxEvaluatorTest { @Test public void testBatchDimension() { - assumeTrue(onnxRuntimeIsAvailable()); + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); OnnxEvaluator evaluator = new OnnxEvaluator("src/test/models/onnx/pytorch/one_layer.onnx"); // Input types @@ -67,7 +63,7 @@ public class OnnxEvaluatorTest { @Test public void testMatMul() { - assumeTrue(onnxRuntimeIsAvailable()); + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); String expected = "tensor<float>(d0[2],d1[4]):[38,44,50,56,83,98,113,128]"; String input1 = "tensor<float>(d0[2],d1[3]):[1,2,3,4,5,6]"; String input2 = "tensor<float>(d0[3],d1[4]):[1,2,3,4,5,6,7,8,9,10,11,12]"; @@ -76,7 +72,7 @@ public class OnnxEvaluatorTest { @Test public void testTypes() { - assumeTrue(onnxRuntimeIsAvailable()); + assumeTrue(OnnxEvaluator.isRuntimeAvailable()); assertEvaluate("add_double.onnx", "tensor(d0[1]):[3]", "tensor(d0[1]):[1]", "tensor(d0[1]):[2]"); assertEvaluate("add_float.onnx", "tensor<float>(d0[1]):[3]", "tensor<float>(d0[1]):[1]", "tensor<float>(d0[1]):[2]"); assertEvaluate("add_int64.onnx", "tensor<double>(d0[1]):[3]", "tensor<double>(d0[1]):[1]", "tensor<double>(d0[1]):[2]"); @@ -99,14 +95,4 @@ public class OnnxEvaluatorTest { assertEquals(expected.type().valueType(), result.type().valueType()); } - private static boolean onnxRuntimeIsAvailable() { - try { - new OnnxEvaluator(simpleModelPath); - return true; - } catch (UnsatisfiedLinkError e) { - System.out.println("onnxruntime not available, test will be ignored"); - return false; - } - } - } |