-
Notifications
You must be signed in to change notification settings - Fork 13
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
High-Speed-Mode TCP Implementation #10
base: master
Are you sure you want to change the base?
High-Speed-Mode TCP Implementation #10
Conversation
@shaun-edwards Could you please either review or allow me rights to merge this PR? |
namespace keyence | ||
{ | ||
|
||
static void worker_main(boost::atomic_flag& flag, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you are using c++11 is there a reason you haven't used the std::atomic_flag?
Doxygen on public methods should be added. |
Has the high speed interface been incorporated into a ROS node(s)? |
Thank you for looking at this. I will get on changes and documentation as quickly as possible. |
@Jmeyer1292 Should this be closed? |
No it's still relevant. This stuff adds the support to reliably sample at 1khz. With the current interface we can only do about 200 Hz. |
Addressing issue #8, this PR adds the ability to interface with the Keyence using the asynchronous "high-speed" mode in which a separate tcp connection is used to constantly stream new profile data.
This PR contains the following:
keyence::HighSpeedTcpClient
that can be started and stopped.This PR introduces the following new dependencies: