Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

hist handling of dimensionality #9390

Closed
timholy opened this issue Dec 17, 2014 · 4 comments
Closed

hist handling of dimensionality #9390

timholy opened this issue Dec 17, 2014 · 4 comments
Labels
needs decision A decision on this change is needed

Comments

@timholy
Copy link
Member

timholy commented Dec 17, 2014

Our hist currently acts more like Matlab's hist, in that when passed a matrix input it operates just along the first dimension. This is different from our other functions like sum, fft, etc. Is that what we want?

hist also only works for vectors and matrices.
@ivarne
Copy link
Member

ivarne commented Dec 17, 2014

Or should we deprecate it (and point to a package) before 0.4 as suggested in #6842?

@ivarne ivarne added the needs decision A decision on this change is needed label Dec 17, 2014
@stevengj
Copy link
Member

And (yuck) we have separate hist and hist2d functions.

Right now, hist seems to be used in RobustStats, ImageView, GoogleCharts, and Distributions.

@ViralBShah
Copy link
Member

We certainly do not need it to work like Matlab.

@ViralBShah
Copy link
Member

Let's have this discussion on #6842.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs decision A decision on this change is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ivarne @ViralBShah @timholy @stevengj