FFT - documentation - examples - linspace construction

cf. https://docs.scipy.org/doc/scipy/reference/tutorial/fft.html

In the examples given in the documentation of FFT shouldn’t the linspace construction in

N = 600
# sample spacing
T = 1.0 / 800.0
x = np.linspace(0.0, N*T, N)

be x = np.linspace(0.0, (N-1)*T, N)

in order for the entries in x to have T spacing?

[peterbell10]

The fft.html is generated from https://github.com/scipy/scipy/blob/master/doc/source/tutorial/fft.rst

That’s correct

I assume the PR has to be made from maintenance/1.5.x branch.

All PRs should be made against master. maintenance branches are only used for back-porting bug fixes.

I’ll also add that linspace(0.0, N*T, N, endpoint=False) might be a clearer way to write it.

[grlee77]

Yes, the file to change is doc/source/tutorial/fft.rst. The PR should be made against the master branch. After merging a maintainer here can backport to 1.5.x as needed (not sure if there are currently plans for an additional 1.5.x release).

There are some contributor guidelines here, but much of the content may not be relevant for a simple documentation fix like this (i.e. no testing, benchmarks, etc. will be involved). There are also docs helping for working with Git.

The beginning sections of the Git docs above list how to create a fork of the SciPy repository and configure an “upstream” remote if you need help with that.