-
Notifications
You must be signed in to change notification settings - Fork 4
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
new example from video stream and video4linux #14
Conversation
Codecov Report
@@ Coverage Diff @@
## master #14 +/- ##
==========================================
+ Coverage 94.23% 95.37% +1.13%
==========================================
Files 4 4
Lines 243 324 +81
==========================================
+ Hits 229 309 +80
- Misses 14 15 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I use it in the same way, but didn't want to include it as an example as it uses an unregistered package (V4L.jl).
If you think its ok, you can go ahead and merge it.
I think there is a lot of new memory being created in the current conversion code. I'm guessing there is room for major speedups with Video4Linux if we use |
Thanks, I think you are correct that there is a lot of room for speedups and any help is much appreciated. |
…/AprilTags.jl into examples/fromvideo
Will certainly help with the speedups. I think it is okay to merge the first example. It would probably be better to move the second LCM message example elsewhere... Maybe directly to |
xref dehann/Caesar.jl#102 |
@dehann, I changed the imshow to avoid the memory leak. It should perform way better now. |
Hi, Thanks! |
can merge if ready. |
cc @GearsAD, for the less leaky memory |
No description provided.