-
Notifications
You must be signed in to change notification settings - Fork 274
Add a CaptureQueue class #580
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
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #580 +/- ##
===========================================
- Coverage 81.16% 68.70% -12.46%
===========================================
Files 51 52 +1
Lines 2814 2841 +27
Branches 328 327 -1
===========================================
- Hits 2284 1952 -332
- Misses 417 782 +365
+ Partials 113 107 -6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Hi @kayoub5 can you see my commit from today ? |
@SebMichaud can you add a test for the CaptureQueue in the Tests folder that uses the class in the way that your labview side is using it? That would let us exercise the api and make it easier for others to help debug and resolve implementation issues. |
Added a CaptureQueue class following issue 575 and its last message from @kayoub5