leezu commented on issue #16878: add micro to pearsonr
URL: https://github.com/apache/incubator-mxnet/pull/16878#issuecomment-559639950
 
 
   The current pearsonr metric is broken, because we cannot actually calculate 
the pearsonr but only an average of batch-wise pearsonrs. We should not ship 
broken features without need. That would be masochistic, deriving pleasure from 
our user's pain inflicted by the broken feature.
   As this PR has very limited scope there should be no concern to include it 
from my perspective. But of course @ptrendx will take the final decision.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to