fixed duplicate calculation of spearmanr function in metrics wrapper.
CONTRIBUTOR
Great, can open a PR in `evaluate` as well to optimize this.

Relatedly, I wanted to add a new metric, Kendall Tau (https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.kendalltau.html). If I were to open a PR with the wrapper, description, citation, docstrings, readme, etc. would it make more sense to do that in the `datasets` or `evaluate` repo (or both)?

Thanks!
2022-07-05T18:24:02Z
https://github.com/huggingface/datasets/pull/4627#issuecomment-1175361030
https://api.github.com/repos/huggingface/datasets/issues/4627
IC_kwDODunzps5GDpYG
https://api.github.com/repos/huggingface/datasets/issues/comments/1175361030/reactions
2022-07-05T18:24:02Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1175361030
https://avatars.githubusercontent.com/u/38060297?v=4
https://api.github.com/users/benlipkin/events{/privacy}
https://api.github.com/users/benlipkin/followers
https://api.github.com/users/benlipkin/following{/other_user}
https://api.github.com/users/benlipkin/gists{/gist_id}
https://github.com/benlipkin
benlipkin
MDQ6VXNlcjM4MDYwMjk3
https://api.github.com/users/benlipkin/orgs
https://api.github.com/users/benlipkin/received_events
https://api.github.com/users/benlipkin/repos
https://api.github.com/users/benlipkin/starred{/owner}{/repo}
https://api.github.com/users/benlipkin/subscriptions
User
https://api.github.com/users/benlipkin
public
CONTRIBUTOR
PR opened in`evaluate` library with same minor adjustment: https://github.com/huggingface/evaluate/pull/176 
2022-07-05T18:37:58Z
https://github.com/huggingface/datasets/pull/4627#issuecomment-1175372775
https://api.github.com/repos/huggingface/datasets/issues/4627
IC_kwDODunzps5GDsPn
https://api.github.com/repos/huggingface/datasets/issues/comments/1175372775/reactions
2022-07-05T18:37:58Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1175372775
https://avatars.githubusercontent.com/u/38060297?v=4
https://api.github.com/users/benlipkin/events{/privacy}
https://api.github.com/users/benlipkin/followers
https://api.github.com/users/benlipkin/following{/other_user}
https://api.github.com/users/benlipkin/gists{/gist_id}
https://github.com/benlipkin
benlipkin
MDQ6VXNlcjM4MDYwMjk3
https://api.github.com/users/benlipkin/orgs
https://api.github.com/users/benlipkin/received_events
https://api.github.com/users/benlipkin/repos
https://api.github.com/users/benlipkin/starred{/owner}{/repo}
https://api.github.com/users/benlipkin/subscriptions
User
https://api.github.com/users/benlipkin
public
MEMBER
> If I were to open a PR with the wrapper, description, citation, docstrings, readme, etc. would it make more sense to do that in the datasets or evaluate repo (or both)?

I think you could just add it to `evaluate`, we're not adding new metrics in this repo anymore
2022-07-07T12:40:59Z
https://github.com/huggingface/datasets/pull/4627#issuecomment-1177549557
https://api.github.com/repos/huggingface/datasets/issues/4627
IC_kwDODunzps5GL_r1
https://api.github.com/repos/huggingface/datasets/issues/comments/1177549557/reactions
2022-07-07T12:40:59Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1177549557
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
During _compute, the scipy.stats spearmanr function was called twice, redundantly, once for calculating the score and once for calculating the p-value, under the conditional branch where return_pvalue=True. I adjusted the _compute function to execute the spearmanr function once, store the results tuple in a temporary variable, and then pass the indexed contents to the expected keys of the returned dictionary.