Argument ordering of toeplitz in t_method="mm"

garrettj403 / CZT

Chirp Z-Transform

MIT License

38 stars 14 forks source link

Argument ordering of toeplitz in t_method="mm" #3

Closed LarsHuebner-LHNW closed 3 years ago

LarsHuebner-LHNW commented 3 years ago

scipy.linalg.toeplitz requires arguments to be ordered (c,r) and not (r,c) in line 70. The error is hidden if M ==N (default for M = None), but for M != N matmul has wrong matrix shape.

garrettj403 commented 3 years ago

Thank you for finding this error.

Fixed in f8d20fc (via your pull request).