search

Planeshifter / node-concept-net

node.js interface to the ConceptNet semantic network API [DEPRECATED; ConceptNet API has changed]
MIT License
29 stars 3 forks source link

Local host with ip and api version support. #2

Closed tissak closed 9 years ago

tissak commented 9 years ago

I am using concept net locally via their docker instance and it creates an ip:port based local host. I modded the constructor (that was dropping the host argument before) to carry host, port, version info. My local docker instance is also up to 5.3 now so I needed that api version info.

Added tests to cover defaults and specified values. Pulled the options building out to a shared function.

My editor stripped trailing white space so the commit looks worse than it is in places. Updated Readme with examples.

Planeshifter commented 9 years ago

Thanks for your pull request! I merged it into the master branch and pushed a new version of the package to npm. Best wishes, Philipp