RNeo4j icon indicating copy to clipboard operation
RNeo4j copied to clipboard

Fixes for CRAN

Open jefferis opened this issue 6 years ago • 7 comments

  • Some minor fixes that should make the package ready for CRAN again
  • main potential issue is if there is still any fallout left from the rust test compilation in #69

jefferis avatar Mar 14 '18 15:03 jefferis

Is the bolt fallback building properly on your system?

Also, I think we're using a lot of deprecated neo4j endpoints. Should we get rid of them before publishing to CRAN?

PlasmaPower avatar Mar 14 '18 16:03 PlasmaPower

bolt fallback seems to be OK now, but there are still some more issues to fix for CRAN. The main one is that there is a method clash with a recommended package:

** checking use of S3 registration ... WARNING
Registered S3 method from a standard package overwritten by 'RNeo4j':
  method     from

this looks like a difficult one to get round without changing the path class inside RNeo4j (it probably wasn't a good choice of class name in the beginning). Current winbuilder results here FYI https://win-builder.r-project.org/eH67Zdbc4lVi/00check.log

jefferis avatar Mar 14 '18 19:03 jefferis

This PR looks like it might pass all CRAN checks. It would be a good idea if anyone interested tried it out:

devtools::install_github("jefferis/RNeo4j@fix/CRAN")

jefferis avatar Jul 19 '18 13:07 jefferis

Just installed on a Linux Mint 18 and R 3.5.0 with no prior installments of RNeo4j, no problems installing. Ran the following demo code from here (substituting username and password), again no errors and the expected output.

graph = startGraph("http://localhost:7474/db/data/", username="neo4j", password="password")

nicole <- createNode(graph, "Person", name="Nicole", age=24)
greta <- createNode(graph, "Person", name="Greta", age=24)
kenny <- createNode(graph, "Person", name="Kenny", age=27)
shannon <- createNode(graph, "Person", name="Shannon", age=23)

r1 <- createRel(greta, "LIKES", nicole, weight=7)
r2 <- createRel(nicole, "LIKES", kenny, weight=1)
r3 <- createRel(kenny, "LIKES", shannon, weight=3)
r4 <- createRel(nicole, "LIKES", shannon, weight=5)

query <- "
MATCH (nicole:Person)-[r:LIKES]->(p:Person)
WHERE nicole.name = 'Nicole'
RETURN nicole.name, r.weight, p.name
"
cypher(graph, query)

query <- "
MATCH (nicole:Person)-[:LIKES]->(p:Person)
WHERE nicole.name = 'Nicole'
RETURN nicole, COLLECT(p.name) AS friends
"
cypherToList(graph, query)

SiggiSmara avatar Aug 03 '18 11:08 SiggiSmara

@PlasmaPower I think you now have write access on this repo. Do you want to merge this PR? It still checks correctly for me – just one NOTE that gnu make is a system requirement.

jefferis avatar Feb 19 '19 15:02 jefferis

Sample r-hub build log for r-release https://builder.r-hub.io/status/RNeo4j_1.7.0.tar.gz-f08ca26e87cd40cf9dbe5ac4c334a1f4

jefferis avatar Feb 19 '19 15:02 jefferis

Note that the check failure above is due to a pre-install neo4j build error on 1 / 4 parts of the build matrix, not to a problem with this package itself.

jefferis avatar Feb 21 '19 19:02 jefferis