Age | Commit message (Collapse) | Author |
|
We've had two reported crashes.
Issue: #468
|
|
With -D_USE_MATH_DEFINES MSVC defines the standard M_PI and
friends.
Since this preprocessor definition is now always passed as part
of the build system for MSVC. We can use M_PI as if on a
mission.
|
|
Suggested-by: MathijsG
Issue: #454
|
|
It's broken and users complain on the issue tracker.
Also fix tab stops.
|
|
Issue: #411
Requested-by: @Tux0Topo
|
|
Reported-by: @andregm3
Closes #409
|
|
This reverts commit c08d63041e184ae642486eebfb4fd770d0a142b6.
Revert "tracker/aruco: allow for more translation on the spline"
This reverts commit f56f30f1d076c8b48a0bd4ce47b26ede618d2880.
|
|
Adjust usages.
|
|
Issue: #411
Reported-by: @Tux0Topo
|
|
Adjust usages in PT and Aruco trackers.
|
|
Issue: #411
Reported-by: @Tux0Topo
|
|
|
|
|
|
We can't depend on M_PI existing after including cmath.
|
|
Sadly, it's only implemented right now on win32.
Remove "set enabled" code for the video widget since it only works for
explicit window minimization, not covering by other windows.
|
|
|
|
|
|
The height was zero on the test video. The ROI check only saw if width
is at least zero. Check for both to be greater than one.
Video provided by: @kblomster
Issue: #375
Also, fix minor issues:
- nix vars that can be const static in function scope
- don't call solvepnp twice where obj_points shift will do
- don't do bounds checking on vector elt access
- respect sprintf varargs type size; change to snprintf for no reason
- fix clamp-to-image logic
- set proper alpha for fps estimation
|
|
Some new matrix element type requirements came up after opencv update
Also,
- switch to matrices of known sizes wherever possible
- split into functions for readability
- use member variables rather than locals to ensure heap allocation
There was also breakage wrt "unknown element type" deep in opencv that
only happens on release mode with no symbols. It's unknown to me
whether the issue got fixed or variable reordering made it corrupt
something else. It appears to work however. -fstack-protector-all
doesn't show any errors at all.
@kblomster says it worked in rc49p2. Looks like -fipa-pta has a
miscompilation despite finally working to begin with.
Reported-by: @kblomster
Issue: #375
|
|
Less overhead this way.
|
|
|
|
- Remove "this->" where it's not needed. Possibly rename shadowed vars.
- Don't reload the options bundle manually since `options::opts' exists
for that very reason.
- Remove '^ \+$' whitespace
- :retab
|
|
In particular PT's calibration interval was 10 Hz which is too much by
far. Make both 4 Hz.
Issue: #344
|
|
libaruco doesn't clamp the min/max sizes but throws an exception.
Reported-by: @duamutefmc
Data-by: @Emton
Issue: #329
Closes #329
|
|
|
|
|
|
Really fast toggling tracking crashed with my PS3 Eye.
|
|
Higher box sizes use more CPU due to the need to convolve a lot.
It looks fine with both high and low exposure on both Logitech C525 and
PS3 Eye webcams.
Issue: #273
|
|
Detection rate stays as good, likely better as before.
@mursey reports in #274 that non-Otsu case eats way more CPU.
|
|
- cycle otsu twice before selecting new box size
don't cycle otsu each frame.
- slower backoff from timeout value before new box size
on occasional detections need at least 5 successes for each failure
to not exceed.
- select more sensible timeout per box size of .35 seconds. this is
enough also with 30 Hz webcams, but too little for 15 Hz modes.
|
|
Closes #224
|