-
Notifications
You must be signed in to change notification settings - Fork 69
Fixes for CRAN #73
base: master
Are you sure you want to change the base?
Fixes for CRAN #73
Conversation
jefferis
commented
Mar 14, 2018
- 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 Add Bolt support #69
* Required for `ifeq ($(shell ...` in Makevars
* MAKE Check was complaining * see https://cran.r-project.org/doc/manuals/r-release/R-exts.html#Using-Makevars
* r-lib/httr/issues/395 was closed in Jul 2017 * MAKE Check doesn't like this
* "figure/" ignores everything underneath the directory but leaves the (empty) directory itself * gave a message during CHECK
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? |
* it's added on build
* another check issue revealed on winbuilder
* to keep check happy
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:
this looks like a difficult one to get round without changing the |
* specifically print.path ** checking use of S3 registration ... WARNING Registered S3 method from a standard package overwritten by 'RNeo4j': method from print.path grid
This PR looks like it might pass all CRAN checks. It would be a good idea if anyone interested tried it out:
|
Just installed on a Linux Mint 18 and R 3.5.0 with no prior installments of RNeo4j, no problems installing. 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 |
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. |