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

Rampdown might possibly be starting one cycle too soon #149

Open
raacampbell opened this issue Feb 6, 2024 · 4 comments
Open

Rampdown might possibly be starting one cycle too soon #149

raacampbell opened this issue Feb 6, 2024 · 4 comments
Assignees
Labels

Comments

@raacampbell
Copy link
Contributor

raacampbell commented Feb 6, 2024

We asked for 800 ms plus 250 ms for Oli's figure and we get the following:
Screenshot 2024-02-06 at 22 52 37

The data are measured with a photodiode and it's possible the start of the rampdown is being missed due to saturation. Over all it is running for the correct time period but maybe we need to tweak how the ramp down happens.

This may be a minor bug or may be a measurement error.

Data from @omgauld001

@raacampbell raacampbell self-assigned this Feb 6, 2024
@raacampbell raacampbell changed the title Rampdown might possibly be starting one cycle too Rampdown might possibly be starting one cycle too soon Feb 6, 2024
@raacampbell
Copy link
Contributor Author

raacampbell commented Feb 6, 2024

I start in simu mode version 1.02 on dev. I load uniAndBilateral_5_conditions.yml then:

[~,~,W]=hZP.sendSamples('cond',2,'stimDurationSeconds',0.8); %Two pos
[~,~,W]=hZP.sendSamples('cond',3,'stimDurationSeconds',0.8); %One pos

The results look as expected.

One stim pos
image

Two stim pos:
Screenshot 2024-02-07 at 00 34 50

Possibly @omgauld001 had an older version with a bug? Or possibly the photodiode was saturated? The lack of the last pulses might be due to the fact that the laser cuts off at very low powers.

@omgauld001
Copy link

We've been using version 1.0.2. in the matlab window it states 'Zapit is up to date'

@omgauld001
Copy link

IMG_9704
This was a measurement taken in Aug 2023. The stim is 500 ms and then 250 ms rampdown. The rampdown starts at the right time, but again it seems like the last laser pulse is not delivered.

@raacampbell
Copy link
Contributor Author

We've been using version 1.0.2. in the matlab window it states 'Zapit is up to date'

You probably are pulling from the dev branch, since master is v1.0.2 (https://github.com/Zapit-Optostim/zapit/releases). Just keep that in mind.

Looks like you had saturation originally and the failure to see the last pulse is just the laser not being able to produce a super-low power.

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