Support positive semi-definite n in fourier_*
- Dominant language
- Python
- Stars
- 226
- Forks
- 56
- PR merge metrics
- No merged PRs in 30d
Description
_From @jakirkham on May 9, 2017 3:13_
Currently `fourier_*` functions only accept `-1` for the `n` argument. Though in SciPy's implementations, it is possible to use positive semi-definite values for `n`. As this wasn't really a priority for the initial implementations, this was merely marked as unsupported. Though it would be nice to be able to provide equivalent support to the SciPy implementation of these functions w.r.t. other values for `n`.
_Copied from original issue: dask-image/dask-ndfourier#5_
Contributor guide
Research direction
Locate the `fourier_*` functions in the Python package and inspect how they currently handle the `n` argument. Compare their behavior with SciPy's Fourier implementations, then verify that positive semi-definite `n` values work equivalently while retaining support for `-1`.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- computer-vision
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100