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

pairwise to work on rows instead of columns. #35

Closed
musm opened this issue Dec 11, 2015 · 3 comments
Closed

pairwise to work on rows instead of columns. #35

musm opened this issue Dec 11, 2015 · 3 comments

Comments

@musm
Copy link

musm commented Dec 11, 2015

Say I have a large matrix X in memory
I would like to use pairwise operating on the rows instead of the columns, without having to pass the transpose of X (so to not allocate another object in memory).

@rawls238
Copy link
Contributor

I think this is a good idea, I'll put up an implementation of this unless @KristofferC has any objections about adding this

@nalimilan
Copy link
Member

transpose now returns a lazy wrapper, so this may no longer be an issue.

OTOH it could be clearer for users to be able to specify the dimension over which variables are stored, since we differ from the convention used by other languages (see also JuliaStats/Clustering.jl#79). Another reason is that cov supports such an argument (and expects variables to be stored by column by default...).

@nalimilan
Copy link
Member

I've filed #121 taking advantage of transpose to support a dims argument. Please comment there.

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

No branches or pull requests

3 participants