-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add spearman correlation #167
Comments
Shall I work on it? |
Sure go ahead :) |
Hello @gautamramk , you're still working on this issue?. If not, then I would like to take over. Yours sincerely |
Hey @NicolasHug, Can you please assign me to the issue? |
Sure. Have you looked at #168? It seems that the bulk of the work is done already and what's left is a benchmark to assess how fast / accurate spearman would be. |
Hmm looks like I cannot assign you to the issue because you don't have write rights. It's not important though so don't worry. |
All right, all right. I see @gautamramk is working with a fork. What would be a good procedure here ? |
Or can the PR be added to another branch in this project so that I can work from my fork ? |
I think a good way to do it would be to pull the PR branch (look here) in your own fork / clone. Then you can either directly work on this local branch of yours, or branch off it to keep it clean (that's what I do). You'll still need to make a new PR though. |
Ah okay, I understand. Thanks for the hint. |
Okay done ! This worked out well:
|
It would be useful to add Spearman Correlation to the similarities module. It is very similar to Pearson's correlation, except a rank is used, instead of ratings.
The text was updated successfully, but these errors were encountered: