Add PearsonCorrelation metric#3212
Conversation
|
@kzkadc thanks for the PR, I haven't yet reviewed it in details, just a remark on the code location which could be better to be at |
|
@vfdev-5 Is it better to create |
|
@kzkadc yes, we'll move all ignite.contrib.metrics into ignite.metrics very soon, so I'd say let's just update your PR such that the metric is in ignite.metrics but we are using |
|
Please also fix code formatting issue |
Head branch was pushed to by a user without write access
|
@kzkadc can you please fix the issue with the implementation using non-cpu device: |
Description: added Pearson correlation coefficient metric to
ignite.contrib.metrics.regression.Check list: