ndarray-stats icon indicating copy to clipboard operation
ndarray-stats copied to clipboard

Published 20 hours ago • verified publisher icon rust-ndarray

Consider renaming `mean` to `arithmetic_mean`.

Open n3vu0r opened this issue 5 years ago • 1 comments

Some functions like mean are implemented in ndarray (here) and ndarray-stats (here). When using SummaryStatisticsExt, the function mean exists two times, which is not a problem but confusing as one implementation returns Option and the other Result and it firstly seems to mismatch with the documentation.

Is it intended to have some implementations duplicated along the two crates on the long run? If so, ndarray-stats could be more specific by renaming mean to arithmetic_mean (and its axis/weighted variants) being consistent with geometric_mean and harmonic_mean. Feel free to close the issue if you prefer a simple mean.

n3vu0r avatar May 14 '20 11:05 n3vu0r

In the long run, ndarray-stats should take over these methods from ndarray IMO.

bluss avatar Apr 06 '21 20:04 bluss