Skip to content

Fix delay at the start of the experiment.#27

Open
nvbln wants to merge 2 commits intomasterfrom
delay_issue
Open

Fix delay at the start of the experiment.#27
nvbln wants to merge 2 commits intomasterfrom
delay_issue

Conversation

@nvbln
Copy link
Copy Markdown
Contributor

@nvbln nvbln commented Mar 28, 2023

Partially fixes issue #24.
Currently the git history is a bit messy. I wanted to test and fix the error using the new changes on other branches. Once they have been merged I'll clear up the history.

@nvbln
Copy link
Copy Markdown
Contributor Author

nvbln commented Apr 4, 2023

Rebased on the current master. Still holds the commits from the other branch. This will be fixed before merging.

@nvbln
Copy link
Copy Markdown
Contributor Author

nvbln commented Apr 18, 2023

After ~420 frames, the following error occurs and the scanning cannot go on:

The application is not able to keep up with the hardware acquisition.
Increasing the buffer size, reading the data more frequently, or specifying a fixed number of samples to read instead of reading all available samples might correct the problem.
Property: DAQmx_Read_RelativeTo
Corresponding Value: DAQmx_Val_CurrReadPos
Property: DAQmx_Read_Offset
Corresponding Value: 0

@nvbln
Copy link
Copy Markdown
Contributor Author

nvbln commented Apr 19, 2023

After ~420 frames, the following error occurs and the scanning cannot go on:

The application is not able to keep up with the hardware acquisition.
Increasing the buffer size, reading the data more frequently, or specifying a fixed number of samples to read instead of reading all available samples might correct the problem.
Property: DAQmx_Read_RelativeTo
Corresponding Value: DAQmx_Val_CurrReadPos
Property: DAQmx_Read_Offset
Corresponding Value: 0

This issue has been solved now, or at the very least it should be pushed towards ~210 000 planes.

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.

1 participant