search

thingdom / node-neo4j

[RETIRED] Neo4j graph database driver (REST API client) for Node.js
Apache License 2.0
926 stars 135 forks source link

dijkstra algorithm support #31

Open flipside opened 12 years ago

flipside commented 12 years ago

I'm looking into using neo4j's built in dijkstra algorithm, it looked like there's already some support in node-neo4j for shortest path but dijkstra support would be great.

aseemk commented 12 years ago

Sorry for the delay @flipside, but great request! We support passing 'dijkstra' as an optional algorithm parameter to the Node::path() method:

https://github.com/thingdom/node-neo4j/blob/0.2.13/lib/Node._coffee#L202

But I didn't realize that full Dijkstra support requires passing an additional cost_property parameter.

I think the signature of the path() method should probably be changed to accept an options hash now instead of a bunch of parameters.

Thanks again — and pull request welcome!

flipside commented 12 years ago

realized i don't actually need this yet but if i do, will take a crack at it.

hackintoshrao commented 10 years ago

This issue has been open from 2 years now !! Thats not good , give me some clue about how to start with it , i could fix this .

aseemk commented 9 years ago

Sorry for the delay @Karthic-Hackintosh.

Good news: it's now possible to achieve this in node-neo4j v2, even though it's not implemented directly, because you can now make arbitrary & custom HTTP / REST API calls (issue #100).

https://github.com/thingdom/node-neo4j/tree/v2#http--plugins

So I'm going to go ahead mark this issue as fixed in v2. Let me know if you have any q's!