The no-stim protocols have at least air-puff on

BhallaLab / MouseBehaviour

Arduino and PointGrey camera based behaviour setup.

https://mousebehaviour.readthedocs.io/en/latest/

GNU General Public License v3.0

2 stars 5 forks source link

The no-stim protocols have at least air-puff on #42

Open ananthamurthy opened 5 years ago

ananthamurthy commented 5 years ago

There were some protocols with no-stim but the air-puff is still turning on at the right time

dilawar commented 5 years ago

This may happen in PROB trials. If these were not PROBE trials then its a bug. If we don't need PUFF in no-stim in PROBE as well, then I need to change the logic.

ananthamurthy commented 5 years ago

Hey!

Yes, no need for probe trials in the no-stim or limited stim protocols..

Ananth

On Tue, Jun 4, 2019 at 1:41 PM Dilawar Singh notifications@github.com wrote:

This may happen in PROB trials. If these were not PROBE trials then its a bug. If we don't need PUFF in no-stim in PROBE as well, then I need to change the logic.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/BhallaLab/MouseBehaviour/issues/42?email_source=notifications&email_token=ACXXJCYPONEFOBIQJPB7SUDPYYPUNA5CNFSM4HR73A2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW3Y7TI#issuecomment-498569165, or mute the thread https://github.com/notifications/unsubscribe-auth/ACXXJCZVAYVUHEFS36KM6EDPYYPUNANCNFSM4HR73A2A .

dilawar commented 5 years ago

Sorry let me be clearer:

When trial is PROBE, then nothing happens in US.
Else US is played.

I understand when CS is NONE and there should NOT be any US (puff/tone) played? Is that right?

dilawar commented 5 years ago

@ananthamurthy ping.

ananthamurthy commented 5 years ago

Hey, Yes, no cs means no puff either.

Additionally, we could have a specific protocol with only puff.

On Tue, 4 Jun 2019 at 2:46 PM, Dilawar Singh notifications@github.com wrote:

@ananthamurthy https://github.com/ananthamurthy ping.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/BhallaLab/MouseBehaviour/issues/42?email_source=notifications&email_token=ACXXJC5PYPMDIR4HDHLUFWDPYYXHHA5CNFSM4HR73A2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW36OEI#issuecomment-498591505, or mute the thread https://github.com/notifications/unsubscribe-auth/ACXXJCYOS7CMTOVFEVMON43PYYXHHANCNFSM4HR73A2A .

dilawar commented 5 years ago

I think the new commit on master should fix it. I am leaving this open, needs testing. I'll it close once you confirm that things are fine.

dilawar commented 5 years ago

They are in config.h file.

/*-----------------------------------------------------------------------------
 *  PINS
 *-----------------------------------------------------------------------------*/
#define         ROTARY_ENC_A                2   // have callback
#define         ROTARY_ENC_B                3   // have callback.
#define         SHOCK_PWM_PIN               5   // PWM
#define         SHOCK_RELAY_PIN             6   
#define         SHOCK_STIM_ISOLATER_PIN     7
#define         TONE_PIN                    8
#define         LED_PIN                     9
#define         CAMERA_TTL_PIN              10
#define         PUFF_PIN                    11
#define         IMAGING_TRIGGER_PIN         13
#define         SHOCK_PAD_READOUT_PIN       A3
#define         SENSOR_PIN                  A5

dilawar commented 5 years ago

Is there any update on this @ananthamurthy ? Just pinging to refresh the old tickets.