| Age | Commit message (Collapse) | Author |
|---|
|
Now that "struct main_settings" doesn't implicitly reload in dtor, we no
longer have to pass it around from the user interface.
Only reload it where it's modified, i.e. in the options dialog. Changing
the filter/dialog/mapping comboboxen implicitly saves the main options
however.
|
|
|
|
The "struct opts" dtor doesn't run at dialog close time so invoke
"reload" on the bundle explicitly.
|
|
- the enumerator and the handle list are now static; joysticks were
erroneously destructed during refresh
- setup dinput only once; creating LPDIRECTINPUT8 many times causes
Release() to invalidate all handles
- don't create joystick handles many times; same thing applies
- refresh joy list on failed acquire; usually joystick is unplugged at
that moment leading to endless stderr spam
- use a static mutex as to avoid dinput races
Issue: #315
|
|
|
|
|
|
|
|
|
|
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
|
|
Issue: #375
|
|
|
|
|
|
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
|
|
Need link dynamic, otherwise symbol lookup failure.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Issue: #318
cf. http://forum.il2sturmovik.ru/topic/2291-open-track/page-26
|
|
|
|
|
|
- switch to our timer since it has more than millisecond precision
- use slider_value for settings
- sanitize allowed scaling curve bounds
- ensure min <= max in dialog
- ensure min <= max at filter runtime
- check for NaN taking -ffast-math into account
- simplify dialog logic
- change delta smoothing constant
|
|
|
|
It needs to filter less for new users.
|
|
|
|
|
|
|
|
Issue: #139
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Qt will take care of its own.
|
|
Issue: #371
Suggested-by: @MathijsG
|
|
Discussed-with: @MathijsG, @Len62
Closes #371
|
|
The reason for its existence is that we can't rotate translation by
rotation as-is. The signs are wrong and we're not using the XYZ order
for Tait-Bryan either.
The line location was incorrect due to mismerge.
|
|
FreeFalcon is gone, long live Falcon!
|
|
Due to memory layout we don't need enable_if for vector.
|
|
It's dubious why that line was there. It needs further testing however.
|
|
Here, the "logic" module has all the stuff for building one's own
graphical user interface.
The "api" module has stuff used by other trackers.
While at it, each of "api", "logic", and "compat" need their own export
headers. This is because of preprocessor symbol clashes.
This is all because a change in the "gui"-only dependency required a
relink of all the trackers, protocols, and flters. It takes too long
when building in the release configuration. With the split, only the
"gui" module gets rebuilt. Since it has close to no static dependencies,
it's fast enough.
|
|
It doesn't work presently.
|
|
|
|
This isn't a proper way to handle the multiple solutions problem. Even
after fixing degree/radian confusion it didn't work.
|
