Closed dogoepp closed 8 years ago
In the meantime, I would appreciate if you can give me some feedback about the utility itself. Is it easy enough to use ? Does it cover your needs ?
I think it has more than what we need... Good job
The missing features are:
management of speed (target and current) instantiation for version 2 of the protocol
We should add this 2, before merging...
Thanks for the feedback. I agree that speed and version 2 are prerequisite to merging. Will take care of it.
Since we've been using this version for a long time already and I didn't have time yet to do the version 2 and speed management, I propose that we merge for now.
I'll handle the missing points in a future pull request. OK ?
Ok On Fri, 9 Sep 2016 at 09:37, Dorian Goepp notifications@github.com wrote:
Since we've been using this version for a long time already and I didn't have time yet to do the version 2 and speed management, I propose that we merge for now.
I'll handle the missing points in a future pull request. OK ?
— You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/resibots/libdynamixel/pull/16#issuecomment-245841115, or mute the thread https://github.com/notifications/unsubscribe-auth/AFu8v45_oDGh44mhnUXVTWqATrUBrVUQks5qoQvLgaJpZM4IP7mJ .
OK
Merging
This version has most of the features we could use on a daily basis.
The missing features are:
I intend to add these features later on.
In the meantime, I would appreciate if you can give me some feedback about the utility itself. Is it easy enough to use ? Does it cover your needs ?
Thanks, Dorian