-
Notifications
You must be signed in to change notification settings - Fork 188
benchmarks: Add a read all available benchmark #883
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
Conversation
|
Results with master branch (not Mike's PR) and a recent DAQmx 26.0 dev build: |
|
Results with #881 and a recent DAQmx 26.0.0 dev build read_all_available went from a mean of 40-60 us to a mean of 28-46 us. |
|
Results with #881 and changing read_all_available went from a mean of 40-60 us to a mean of 3 ms. |
35e5700 to
2b90a49
Compare
Test Results 42 files 42 suites 1h 2m 7s ⏱️ Results for commit e2efe97. ♻️ This comment has been updated with latest results. |
I've updated CHANGELOG.md if applicable.I've added tests applicable for this pull requestWhat does this Pull Request accomplish?
Add AI read benchmarks for READ_ALL_AVAILABLE
Why should this Pull Request be merged?
I want to understand the effect that #881 has on performance.
What testing has been done?
Ran new test case along with read_many_sample.