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

Add protocol for delay conditions #61

Open
hrishi16 opened this issue Aug 27, 2019 · 7 comments
Open

Add protocol for delay conditions #61

hrishi16 opened this issue Aug 27, 2019 · 7 comments
Assignees
Labels

Comments

@hrishi16
Copy link
Contributor

Changing the present csv doest take care of this.
IMG_20190827_170000

A protocol format neds to be added such that the trace interval is not there, and there is an overlap of the 2 stimuli (CS and US. They co-terminate.
Variations can be made in duration of the stimuli.

@dilawar
Copy link
Contributor

dilawar commented Aug 29, 2019

This is possible without using timers as well. I'll send a PR over the weekend.

@dilawar dilawar added the TODO label Aug 29, 2019
@dilawar
Copy link
Contributor

dilawar commented Sep 26, 2019

Will there be PROBE trial in this? If there is any PROBE, What is the behaviour of PROBE?

@hrishi16
Copy link
Contributor Author

hrishi16 commented Sep 26, 2019 via email

@dilawar
Copy link
Contributor

dilawar commented Sep 26, 2019

In current revision, following is allowed.

Overlapping CS/US. Following two cases are allowed.

   +----------------------------------------------+
   |                +-----------------------------+                                       
   |                |                             |
--+ ----------------+                             +-------------------
   +-------------------------------------------------------+
   |                +-----------------------------+        |
   |                |                             |        |
--+ ----------------+                             +--------+----------

US is not completely inside CS: NOT ALLOWED.

   +-------------------------------------+
   |                +--------------------|---------+           
   |                |                    |         |            
--+ ----------------+                    +---------+------------------

If the last one is required, this will require a major rewrite/refactoring. If not, then we are almost done.

dilawar pushed a commit that referenced this issue Sep 26, 2019
@dilawar
Copy link
Contributor

dilawar commented Sep 26, 2019

There will be probe. The probe trial will only have the CS (tone or light) and no US

On Thu, 26 Sep, 2019, 11:26 Dilawar Singh, @.***> wrote: Will there be PROBE trial in this? If there is any PROBE, What is the behaviour of PROBE? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#61?email_source=notifications&email_token=AHVPH2P7YQR6Y6HBBTMPMCDQLRFJFA5CNFSM4IQDMQ62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7UMRIY#issuecomment-535349411>, or mute the thread https://github.com/notifications/unsubscribe-auth/AHVPH2OLFPZG2CPJZ6DBQADQLRFJFANCNFSM4IQDMQ6Q .

Ok. PROBE is allowed.

@dilawar
Copy link
Contributor

dilawar commented Sep 27, 2019

@hrishi16 Have a look when you get time.

@dilawar
Copy link
Contributor

dilawar commented Oct 9, 2019

$ cd MouseBehaviour
$ git pull
$ git branch #to see which branch you are on; mentioned by * in front of it.
$ git checkout overlap_cs_us_issue_61   # overlap branch for issue 61 

Rest is the same.

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

No branches or pull requests

2 participants