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.

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

ENH: pdist option to return squareform/dense array #7984

Open
jakirkham opened this issue Oct 6, 2017 · 1 comment
Open

ENH: pdist option to return squareform/dense array #7984

jakirkham opened this issue Oct 6, 2017 · 1 comment

Comments

@jakirkham
Copy link
Contributor

Would be nice if pdist took an optional flag to return the dense square array (equivalent to running squareform) instead of sparse 1D array.

@tylerjereddy
Copy link
Contributor

Considering the relative ease with which once can call squareform on the result, I'm not entirely clear on the advantage of adding the flag.

The condensed matrix can be quite a bit smaller / more efficient and requiring the user to explicitly use another function to expand to a (potentially much larger) redundant distance matrix doesn't seem like that bad of an idea to me, but others may certainly disagree.

I think there have recently been quite a few changes / attempts to clean up the bloated arguments for pdist and cdist as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants