RNeo4j
RNeo4j copied to clipboard
Fixes for CRAN
- 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
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?
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
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")
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)
@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.
Sample r-hub build log for r-release https://builder.r-hub.io/status/RNeo4j_1.7.0.tar.gz-f08ca26e87cd40cf9dbe5ac4c334a1f4
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.