small fix to doc for linspace #14526
Conversation
linspace is defined https://github.com/JuliaLang/julia/blob/master/base/range.jl#L235 to have a default of n=50 points, not n=100. Not sure if I need to do more for this.
|
I wonder if 100 should be the real default in the code instead. |
|
Also needs to be changed in the corresponding rst signature. Thanks! |
|
Okay, I updated the .rst file. Is there some way to squash commits through the GitHub interface? I too think the default should be 100 and not 50, as that is the MATLAB users expectation, but this just changes the docs to match the code. |
|
FWIW numpy uses default = 50. |
|
I would just deprecate the method with the default value, as it makes the code harder to understand given that the default is totally arbitrary and changes depending on the language. |
|
I think most codes that use linspace without specifying a value don't really care if the default is 100 or 50. Usually you just use it if you want a "somewhat fine" discretisation of an interval for, e.g for plotting. -1 for deprecating it. |
|
I tried to squash, but ended up creating a new PR (#14528). Closing this and will merge the other one. |
linspace is defined https://github.com/JuliaLang/julia/blob/master/base/range.jl#L235 to have a default of n=50 points, not n=100. Not sure if I need to do more for this.