neo4j-graph-analytics / ml-models

Machine Learning Procedures and Functions for Neo4j
https://github.com/neo4j-graph-analytics/ml-models/releases/tag/1.0.0
Apache License 2.0
64 stars 23 forks source link

Build failures with incorrect procedure names #8

Open bradnussbaum opened 5 years ago

bradnussbaum commented 5 years ago

Results :

Failed tests: shouldOnlyKeepFeaturesThatAddSomethingUnique(embedding.PruningTest): expected:<[9.0000, (..) pruneEmbeddingsAndFeatures(embedding.PruningTest): arrays first differed at element [3]; expected: but was: makeModel(regression.LogisticTest): unable to read csv file for test data

Tests in error: stream(embedding.DeepGLIntegrationTest): There is no procedure with the name algo.deepgl.stream registered for this database instance. Please ensure you've spelled the procedure name correctly and that the procedure is properly deployed. write(embedding.DeepGLIntegrationTest): There is no procedure with the name algo.deepgl registered for this database instance. Please ensure you've spelled the procedure name correctly and that the procedure is properly deployed.

Tests run: 33, Failures: 3, Errors: 2, Skipped: 0

[INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 16.798 s [INFO] Finished at: 2018-10-20T21:40:28-04:00 [INFO] ------------------------------------------------------------------------ [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project neo4j-ml-models: There are test failures. [ERROR] [ERROR] Please refer to /Users/bradnussbaum/git/neo4j-graph-analytics/ml-models/target/surefire-reports for the individual test results. [ERROR] -> [Help 1] [ERROR] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch. [ERROR] Re-run Maven using the -X switch to enable full debug logging. [ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

bradnussbaum commented 5 years ago

This is build from latest master branch.

mneedham commented 5 years ago

Yeh we changed the name of the procedure and I thought I'd updated the tests, but maybe I didn't push it or something. I'll check that other test too - perhaps an unnecessary ordering.