search

baalexander / rosnodejs

A ROS client library using node.js
http://baalexander.github.com/rosnodejs/
MIT License
36 stars 8 forks source link

Dynamic message parsing #23

Closed temsa closed 12 years ago

temsa commented 12 years ago

This is our implementation for dynamic message parsing. Parsing was pretty fast, although not really tested, I'll let you try to have numbers.

Sorry for the delay, it was pretty complicated these last days, we have received what will be a robot in the team, and had to focus on making it work. So @maxired from my team had to finish some parts of the initial work, and it is pretty well tested.

The ros.types API is pretty easy to make on top of this. I'll discuss this topic with you in the dedicated bug

baalexander commented 12 years ago

Thanks @temsa and @maxired!

I'll take a deeper look tomorrow as I start moving rosnodejs to the API mentioned in #19 (I just posted an update, fyi).

baalexander commented 12 years ago

I merged the changes into the 0.1.0-wip branch. I went through the test file first (thanks for the test cases, by the way). I hope to make it through messages.js within the next couple days.

I am closing the pull request since the changes have been merged, but I will post any questions / comments here after I get through messages.js.