-
Notifications
You must be signed in to change notification settings - Fork 5
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
Issue1 c++ zmq reader #2
base: master
Are you sure you want to change the base?
Issue1 c++ zmq reader #2
Conversation
Hello @anjaldoshi I have some questions/suggestions about the ZMQ node in Open-Ephys.
Thanks ! Have a good day. Note: I tried to follow your style in the code but tell me if you have a favorite style/syntax that I am not following. |
Hi Marine, As I'm not the one who developed this plugin, I won't be able to answer all your questions.
I think the only major thing you should focus in the example is receiving continuous data from Open Ephys via ZMQInterface to your client app and then displaying/processing that data in your app. Also, regarding the current status of your example, I see that your have |
Hello @anjaldoshi For the other questions, in that case, my example is finished. I did not expect to spend time on an GUI to look at the signal. |
I just added the feature to show a list of connected apps in the ZMQInterface's editor. It should show "No app connected" when you drop the plugin in the signal chain initially. As soon as you connect an app and start acquisition in Open Ephys GUI, the app name should show up in green text in the list as shown in the image below. If, for some reason, the app disconnects in between, the name of the connected app turns red after a while. You can test this feature out in the branch available here: https://github.com/open-ephys-plugins/ZMQInterface/tree/list-connected-apps Regarding your example, I tried running following the steps you mentioned in the Readme (except running in a conda/pip environment), but I get the following error when I run cmake:
|
Hello @anjaldoshi,
I will modify the requirement accordingly. Thanks for the catch. |
Happy new year ! |
Happy New Year, Marine! The reason this PR has not been merged yet is because we want to add support for Windows & Mac for the ZMQ Reader. I'm planning to work on it soon but I have some other things that I need to finish before I can get back to this PR. If you have the time and would like to work on this then that's great, otherwise I've it on my to-do list anyway. |
Example - ZMQ reader project in C++
TODO:
[x] Create socket connections
[x] Parse data and filled buffer
[x] Parse event/spike message
[x ] Send heartbeat
[x ] Doc
close #1
@anjaldoshi