Skip to content
This repository has been archived by the owner on May 23, 2019. It is now read-only.

Fixes for CRAN #73

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Fixes for CRAN #73

wants to merge 12 commits into from

Conversation

jefferis
Copy link

  • 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
* 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
@PlasmaPower
Copy link
Collaborator

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
@jefferis
Copy link
Author

jefferis commented Mar 14, 2018

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

* specifically print.path

** checking use of S3 registration ... WARNING
Registered S3 method from a standard package overwritten by 'RNeo4j':
  method     from
  print.path grid
@jefferis
Copy link
Author

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")

@SiggiSmara
Copy link

SiggiSmara commented Aug 3, 2018

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)

@jefferis
Copy link
Author

@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
Copy link
Author

@jefferis
Copy link
Author

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants