nathanntg/ccontour

Consensus Contours

ccontour's Issues

Add tests

Bundle compiled mex file

Since there is a single target platform, it would be nice to include a compiled MEX file with the repository.

Add time and frequency outputs

For plotting, it is valuable to know the time and frequency bins that correspond with rows and columns. Those should be accessible via both the C API (maybe using the dimensions structure and the setup), and should be potentially accessible as the second and third output arguments from the MEX file. Such as:

[..., F, T] = ccontour(signal, fs);

Does not correctly calculate FFT

Given how the Accelerate framework calculates the FFT, the first entry in the result combines the 0th bin (DC) and the Nth bin (where N is the size of the FFT transform). These values are stored in the real and imaginary part respectively.

The correct result should be a vector of N+1 values, by correctly separating these two values.

See example of how this is calculated in a newer project:

https://github.com/nathanntg/bela-warp-detect/blob/master/BelaWarpDetect/Library/CircularShortTimeFourierTransform.cpp#L285-L288