opentrack.git/tracker-aruco/ftnoir_tracker_aruco.cpp, branch opentrack-2.3-rc49p10head tracking software for Windows and Linuxhttp://ananke.misaki.pl/cgit/opentrack.git/atom?h=opentrack-2.3-rc49p102016-07-31T11:47:09Ztracker/{pt,aruco}: don't include opencv highgui, videoio is enough2016-07-31T11:47:09ZStanislaw Haliksthalik@misaki.pl2016-07-31T11:47:09Zurn:sha1:ccafc5f054733e883bb0ef8b0c698b4c9c86fff9some: replace hardcoded pi values with the same pi constant2016-07-19T05:17:22ZStanislaw Haliksthalik@misaki.pl2016-07-18T10:14:51Zurn:sha1:a9b804e18c811c781a099a70b960f5bbde61257d
We can't depend on M_PI existing after including cmath.
gui, tracker/{aruco,pt}, api: detect whether widget is visible on screen2016-07-16T21:54:27ZStanislaw Haliksthalik@misaki.pl2016-07-16T21:44:31Zurn:sha1:754ae1a54132eb41332267fc70a42595017c5a6e
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.
tracker/aruco: undo corner refinement method switch2016-07-08T13:11:42ZStanislaw Haliksthalik@misaki.pl2016-07-08T13:11:42Zurn:sha1:5adc4acd119923278ed346ceb0c899751c5c1a50tracker/aruco: pre-initialize pose with DLS for LM2016-07-08T12:32:17ZStanislaw Haliksthalik@misaki.pl2016-07-08T12:32:17Zurn:sha1:29721d11813a97918242c08238c97b1dd9708740tracker/aruco: fix #3752016-06-27T12:44:35ZStanislaw Haliksthalik@misaki.pl2016-06-27T12:44:35Zurn:sha1:10c47b159d5ea0e3888318d801be065b6d921b9d
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
tracker/aruco: fix crash after opencv update2016-06-24T13:47:22ZStanislaw Haliksthalik@misaki.pl2016-06-24T07:09:22Zurn:sha1:8b9f88d4ed1482ed508826801e559962970dbc96
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
tracker/{aruco,pt}: disable opencv multithreading2016-06-14T16:14:46ZStanislaw Haliksthalik@misaki.pl2016-06-14T09:45:53Zurn:sha1:a8da0d997048007d3feb2e3814ba93bbfd4d4ef8
Less overhead this way.
tracker/aruco: unsigned where able2016-06-14T16:14:46ZStanislaw Haliksthalik@misaki.pl2016-06-12T14:51:56Zurn:sha1:6b7e7f988d58e84a9e051fd6a40ed4535f73e687many modules: trivial cleanups only2016-05-26T15:19:19ZStanislaw Haliksthalik@misaki.pl2016-05-26T15:09:17Zurn:sha1:b008aefee41a21b24f1d2f1f23cd6c78b888ede5
- 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