[GitHub] [spark] srowen commented on issue #24478: [Spark-27584][MLlib] Add 'Mean reciprocal rank' to RankingMetrics

2019-05-10 Thread GitBox
srowen commented on issue #24478: [Spark-27584][MLlib] Add 'Mean reciprocal 
rank' to RankingMetrics
URL: https://github.com/apache/spark/pull/24478#issuecomment-491407736
 
 
   @holdenk yeah fair point. Right now the .ml implementation doesn't already 
support meanAveragePrecision or anything. I think that could be a fine separate 
PR.


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:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] srowen commented on issue #24478: [Spark-27584][MLlib] Add 'Mean reciprocal rank' to RankingMetrics

2019-04-28 Thread GitBox
srowen commented on issue #24478: [Spark-27584][MLlib] Add 'Mean reciprocal 
rank' to RankingMetrics
URL: https://github.com/apache/spark/pull/24478#issuecomment-487390405
 
 
   I don't think this is important enough to add... .mllib is a legacy API 
anyway.


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:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org