WIP: Allow Driver as argument to from_neo4j

[adamnsch]

Thank you for your contribution to the Graph Visualization for Python project by Neo4j.

Before submitting this PR, please read Contributing to the Neo4j Ecosystem.

Make sure:

• You signed the Neo4j CLA (Contributor License Agreement) so that we are allowed to ship your code in our library
• Your contribution is covered by tests

[adamnsch]

Lets you do:

with GraphDatabase.driver(URI, auth=auth) as driver:
    VG = from_neo4j(driver)
    VG.render()

[FlorentinD]

i think if we allow a driver here, we should also allow to specify a query here instead of the row_limit.
Otherwise we end up with building a Cypher query builder

[adamnsch]

I don't think I agree with that. If they want to do their own query they can do it first, and then just provide the Result. The idea of this is to make it so that they don't even have to make a query - that's the added value

[FlorentinD]

okay if thats the main part, how about just passing the url + auth instead and we construct the client as well?

I think its always a bit to remember how to construct the driver (>0s)

[adamnsch]

That's a cool idea! I'll try it

[adamnsch]

What I don't like is that it requires more parameters

[FlorentinD]

you can have a dataclass to combine url + auth. gds has DbmsConnectionInfo now.