tobac icon indicating copy to clipboard operation
tobac copied to clipboard

Default values of the histogram bins in analysis

Open snilsn opened this issue 1 year ago • 2 comments

I just clicked on the API reference in the documentation and noticed that the default bin edge values for all the histogram functions are these really long, hand-coded lists of numbers. This is not really a high priority, but it just doesn't look professional and could be done much more compactly with np.arange().

I don't know if these functions are actually used that much, and maybe I'm the only one who's annoyed by this, but it's one of the first things you see when you look at the API.

snilsn avatar Nov 14 '23 11:11 snilsn

I agree, this also sticks out to me when checking the API reference! Best practice would be to make the default value None and then create the array inside the function, rather than have a mutable default keyword value

w-k-jones avatar Nov 18 '23 15:11 w-k-jones

Hi. I would like to work on this.

martha889 avatar Nov 29 '23 14:11 martha889