Skip to content
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

Timestamp update #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

markschatza
Copy link
Contributor

Hello again!

This has a few updates:

  1. Events now show up in LFP Viewer.
  2. Doing this required lining up the timestamp of the logic-gate event to the timing of the second at a later event.

Let me know if you have any comments!

@alejoe91
Copy link
Contributor

alejoe91 commented May 4, 2020

@markschatza Thank you so much for this! :)

I will test everything between today and tomorrow (and propagate this change also to the Tracking Stimulator)!

@alejoe91
Copy link
Contributor

alejoe91 commented May 4, 2020

@markschatza it seems to be crashing on mac, using the File Reader as a continous source, the sync plugin to generate the initial TTL, and the Logic gate to delay it. I tested both the AND and the DELAY. I'll try it on linux in the next days.

Could this be due to the Sync port?
Would you have time to test that as well?

Thanks again!
Alessio

@markschatza
Copy link
Contributor Author

Yeah, using the Sync port could be an issue. Let me look into this (most likely tomorrow). It probably isn't playing nice with the events coming from two different sources.

Can't imagine it is a mac issue, but maybe!

@alejoe91
Copy link
Contributor

alejoe91 commented May 4, 2020

Thanks! Well testing on Linux wouldn't hurt ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants