-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do we need different bindings definitions for each target language? #32
Comments
Another example is |
(Just my two cents) Good question. While I see benefits in having an API that feels "native" to the language in question, I would stick to whatever SWIG produces based on our C++ API. And for the C++ API, I would use what makes more sense for C++. Basically, I wouldn't worry too much about this for now and defer the problem to when we find an issue in one language, or we see that one particular language is important enough as to provide custom API for it (outside what SWIG provides). |
Yeah I think that's a good strategy for our v0 at least! Focusing on one language at a time will help keep things reasonably scoped, especially when there are so many other variables. |
Alice brought this up in #105 and it got me thinking about it again, but I just came to Edu's conclusion. I would be a far amount of initial work to write extra interface files, and I want to start actually using the library for testing :) |
Do we want in the python and node bindings an option to return a JSONifiable object or JSON object that represents the nodes -- with all the properties we can find on that object? We could use getters in javascript (do they have this in python?). This might be slightly more annoying to do in c++. |
Python, JavaScript etc. have quite different conventions as to how things are named, whether things are properties or getter-methods etc. Do we want to explicitly have API definitions which are specific to each target language?
snake_case
for function names in Python andcamelCase
for function names in JavaScript?The text was updated successfully, but these errors were encountered: