-
Notifications
You must be signed in to change notification settings - Fork 8
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
Why deprecated #37
Comments
Hi @patrick-radius, thanks for the question. The main reason this project was deprecated was a timing issue. I had not finished implementing the entire ROS protocol in rosnodejs when I changed jobs, so I decided it was best to deprecate the project instead of let it linger in limbo. While I don't think there was anything fundamentally wrong with the rosnodejs module, I'm sure it could be done better today with improvements in Node, ES6, coding practices, etc. For what it's worth, I receive emails semi-frequently from people interested in a native Node.js client library for ROS. I still think it would be useful to the community if someone wants to start a new project. |
Hi @baalexander thanks for your clear answer. Anyway, would you accept pull-requests to this repository, or should somebody else start from scratch? |
roslibjs is certainly more feature complete and has a strong community behind it. I believe Node.js support was added, but still calling rosbridge as opposed to being a pure-JS native client library. I haven't verified though. As for where to contribute, let's leave this repo deprecated. If you create a new project that reaches full client library support using only Node, I'll be happy to link to it from rosnodejs' README. I can't really say Best of luck! |
Hi @baalexander, I've taken it on to complete the work you started -- thanks for getting all the hard stuff done already! My repo is https://github.com/chfritz/rosnodejs. So far I fixed a bug that prevented subscribing to message types that used the Header type, and added the ability to call services. I am wondering how to proceed regarding the name. Do you want to grant me rights to publish updated npm packages for rosnodejs or would you prefer if I changed the name to something new? I would prefer the former -- mostly because I think it's the best possible name -- but can go either way. Please let me know. Best, |
Hi @chfritz, great to hear from you. A few questions:
I ask because I'm not convinced rosnodejs is a good foundation to build on. It's out of date, my Node.js coding abilities were lacking, etc. Rethink's rosjs or Robot Web Tool's roslibjs could be better starting points. |
oh wow, I had no idea rosjs existed. I guess this is very recent development, too. Thanks for mentioning this. I'll have to try that out. So far all I see in the code looks very good and clean. We are pretty committed to finding a solution that is better than rosbridge + roslib for server side integration of ros and node, and it would be a team in the long run but initially mostly me working on it. If we end up using this in production we would obviously maintain it. But seeing how clean rosjs is with ES6 classes and everything, I'm going to check that out first and potentially join forces with Rethink. Thanks for the recommendation! I'll let you know if for whatever reason that path doesn't work out for us and we'd like to consider rosnodejs again. |
Hi,
i was wondering why this repository was deprecated.
Is this mostly an issue about time or change of responsibilities or is there something fundamentally 'wrong' about this implementation?
I am aware of the other work being done like the rosbridge etc, but i am also very interested in running a real JS based implementation on the robot target itself, and i think especially in combination with something like the Tessel 2 this can be really powerful.
The text was updated successfully, but these errors were encountered: