Hi,
In the following two lines, I guess we need to avoid including two of the same points ( 0 = np.pi*2 )?
|
z = np.linspace(0, np.pi * 2, n) |
|
z = np.linspace(0, np.pi * 2, n) |
z = np.linspace(0, np.pi * 2, n)
to
z = np.linspace(0, np.pi * 2, n, endpoint=False)
In the reposition part (
|
for i in range(n_samples): |
),
I guess it makes sense not to include the edge points twice.
Please let me know if I am wrong. Thank you!
Hi,
In the following two lines, I guess we need to avoid including two of the same points ( 0 = np.pi*2 )?
pycircular/pycircular/circular.py
Line 39 in d306f15
pycircular/pycircular/circular.py
Line 88 in d306f15
z = np.linspace(0, np.pi * 2, n)to
z = np.linspace(0, np.pi * 2, n, endpoint=False)In the reposition part (
pycircular/pycircular/circular.py
Line 48 in d306f15
I guess it makes sense not to include the edge points twice.
Please let me know if I am wrong. Thank you!