Correcting clang error in Travis CL

Merge branch 'master' into rotation

Merge remote-tracking branch 'upstream/master' into rotation

Big batch does not work.

Unify init messages and add FFT size message

Use fixed format for output numbers

test: Show logs when not running on Travis CI

Add book sequence to tests

Add useful settings into Makefile (as comments)

Allow CUDA autodetection with cmake < 3.7

Autodetect CUDA architecture

Do not require ifdefs when using sqr_norm()

Revert FHoG::extract changes

It's not necessary for this function to know about our threads etc.

Simpify init

Simplify sub_grid_scale

Remove unnecessary typecasting

Replace (p_windows_size.* / p_cell_size) with p_roi and remove useless typecasting

Use cv::Size for p_roi

Fix scaling condition

"Or" makes more sense than "and".

Simplify fit scale computation, cleanup

Fix scale initialization

Without this, estimation of tracked object scale doesn't work
correctly.

This should have been done in commit 495214a ("Move scale and
async_res variables to ThreadCtx", 2018-09-12).

Allow compiling all variants with ninja

It is faster - see the comment in the Makefile.

Travis complains about long logs

travis: Run tests

We run them only with clang. This has two reasons:

1) We cannot run CUDA and this job does not build cuda.
2) It is the fastest job, so tests will be run while other jobs will
   still compile.

Test improvements

- we test --fit flag
- we do not rerun tests if the log is up to date
- vot2016.zip is not downloaded in vot2016 directory already exists

Allow getting filter response

It roughly corresponds to tracker's view of tracking accuracy. This value
is produced even when ground truth data is not available.

PITOM wanted to have this for their drone use case.

Fixes #19.

Add automated testing

Merge README update

Update README

Move scale and async_res variables to ThreadCtx

Change p_threadctxs from list to vector

Use list of thread contexts rather than list of pointers to thread contexts

This simplifies some constructs in the code.

Remove useless typecasting

Reformat

Improve error message

clang-format: Always move "if" body to a separate line

Merge branch 'cuda-streams'

Merge branch 'rotation' of https://github.com/Shanigen/kcf into rotation

Change the float to text to be more clean and easier to read.

Removed scale parameter from KCF_Tracker::geometric_transformations.

Renamed search parameter in KCF_Tracker::geometric_transformations to allow_debug.

Integration of ThredCTX to rotation.

Fix clang warnings

Mostly unused variable/function.

Added comments to variables in ThreadCtx + renamed p_scale_vars in kcf

This commit adresses issue #11.

Correction for CUFFT Big batch OpenMP

Corrected warnings and errors in CUFFT version

These problems were found by Travis CL.

Corrected warnings in CPU ComplexMat

This fixes #15.

Do not force CUDA_HOST_COMPILER to g++

cmake sets CUDA_HOST_COMPILER automatically based on CC environment
variable. If one uses custom CC variable and CUDA_HOST_COMPILER is set
this way, build usually fails. This happens, for example, when one
needs to use an older compiler (e.g. gcc-5), because the newer default
compiler is not supported by CUDA.

This fixes #10.

Streamlined ThreadCtx

Removed obsolete variables and variables that were
only used in zero index and moved them to kcf.

Minor CMake correction for CUDA versions

Removed obsolete include in CUFFT

CUDA version of the tracker no longer uses OpenCV's CUDA
API.

Added DynMem class

DynMem class streamlines dynamic memory allocation for both CUDA
and CPU version of the tracker. Also some minor changes in fft_cufft.hpp
were made, because there was a problem with order of header includes.

Made dynamic memory allocation more readable

Added geometric_transformations function

Addded visual debug mode and also modified the rotation tracking implementation.

Added visual debug output.

Commiting work done so far on rotation

Saving work done so far on rotation.

Minor correction

Again corrects error with Clang 3.8 in Travis CL

Fixes Clang 3.8. error in Travis CL

First error of Clang 3.8 on Travis CL

Same as commit before correcting dbae3ced2d9959a6552845101b46989f79d697cc

Corrects small mistake from dbae3ced2d9959a6552845101b46989f79d697cc

travis: Also Build with clang-5

travis: Also build with clang-3.8

travis: Build with -Werror

travis: Compile with gcc-5

TX2 has also gcc-5. The advantage is that this gcc version does not
produce some (false?) warnings about uninitialized variables.

travis: Use APT addon rather than a bunch of sudo commands

This looks like a cleaner approach.

travis: Use less CUDA packages in dependencies

This makes Travis checks faster (from 5:00 to 3:30)

Added comments to variables in ThreadCtx + renamed p_scale_vars in kcf

This commit adresses issue #11.

Correction of commit 079fbb857002b7f759fbcf7f3aa5d9d8549de31e

Correction for CUFFT Big batch OpenMP

Corrected warnings and errors in CUFFT version

These problems were found by Travis CL.

Corrected warnings in CPU ComplexMat

This fixes #15.

Add markers to compile output

With this, it will be easier to figure in which build variant an error
happens.

Enable Travis CI

Do not force CUDA_HOST_COMPILER to g++

cmake sets CUDA_HOST_COMPILER automatically based on CC environment
variable. If one uses custom CC variable and CUDA_HOST_COMPILER is set
this way, build usually fails. This happens, for example, when one
needs to use an older compiler (e.g. gcc-5), because the newer default
compiler is not supported by CUDA.

This fixes #10.

Changed variables in ThreadCtx to uint

Variables which are always positive were changed to uint.
This meant that some changes had to be made to ComplexMat
to reduce the number of typecasts.

Streamlined ThreadCtx

Removed obsolete variables and variables that were
only used in zero index and moved them to kcf.

Changed p_windows_size and windows_size type to cv::Size

Minor CMake correction for CUDA versions

CUFFT now uses ThreadCtx and also pragmas from pragmas.h

Removed obsolete include in CUFFT

CUDA version of the tracker no longer uses OpenCV's CUDA
API.

Renamed Scale_Vars to ThreadCtx

Also part of this commit is deletion of forward declaration of
Scale_Vars (now ThreadCtx) in fft_fftw.h, fft_opencv.h and fhog.hpp.

Added DynMem class

DynMem class streamlines dynamic memory allocation for both CUDA
and CPU version of the tracker. Also some minor changes in fft_cufft.hpp
were made, because there was a problem with order of header includes.

Moved omp pragmas declaration to seperate file

Definition of omp pragmas have nothing to do with Scale_vars,
for that reason pragmas.h was created with all of the definitions.

Corrects bug in 7a7ae99a73cc8456950d73e83586efbff6c28379

Revert "Removed obsolete method set_stream"

This reverts commit a22c75434c0db289b63de882fc1677108882b445.

Made dynamic memory allocation more readable

Removed obsolete method set_stream

In CUDA version of ComplexMat class was unused set_stream method.

CMake cleanup

Removed redundant forward declaration.

This commit fixes #13.

Formatted files with Clang Format.

Reformat some parts with clang-format

Big Batch mode now support OpenMP

Big batch mode now can use OpenMP, while using
the Scale_vars structure.

CUDA streams works

This commit adds support for CUDA streams and also corrects issue
with CUFFTW version of the tracker. All version now works without
any problem. CUDA streams version currently does not support C++
async directive and only OpenMP.
TODO: Implement correct use of OpenMP in big batch mode.

Work done so far on CUDA streams

This commit contains not only workd done to implement CUDA streams,
but also some changes to correct warnings and make the naming little
bit more consistent.

Big batch mode now works for all versions

Removed OpenCV_CuFFT version

Removed obsolete version of the tracker, which does not exist, but
was still available in CMake.

FFT does not take whole Scale_vars struct

FFT class now does not need to know anything about the tracker. It only gets as
input parameters objects and variables on which to perform FFT. Also Scale_vars
constructor was added, which allocates memory for variables.