{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/20629","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/20629/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/20629/comments","events_url":"https://api.github.com/repos/apache/spark/issues/20629/events","html_url":"https://github.com/apache/spark/pull/20629","id":297851993,"number":20629,"title":"[SPARK-23451][ML] Deprecate KMeans.computeCost","user":{"login":"mgaido91","id":8821783,"avatar_url":"https://avatars1.githubusercontent.com/u/8821783?v=4","gravatar_id":"","url":"https://api.github.com/users/mgaido91","html_url":"https://github.com/mgaido91","followers_url":"https://api.github.com/users/mgaido91/followers","following_url":"https://api.github.com/users/mgaido91/following{/other_user}","gists_url":"https://api.github.com/users/mgaido91/gists{/gist_id}","starred_url":"https://api.github.com/users/mgaido91/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgaido91/subscriptions","organizations_url":"https://api.github.com/users/mgaido91/orgs","repos_url":"https://api.github.com/users/mgaido91/repos","events_url":"https://api.github.com/users/mgaido91/events{/privacy}","received_events_url":"https://api.github.com/users/mgaido91/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":16,"created_at":"2018-02-16T16:54:40Z","updated_at":"2018-04-23T12:44:13Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/20629","html_url":"https://github.com/apache/spark/pull/20629","diff_url":"https://github.com/apache/spark/pull/20629.diff","patch_url":"https://github.com/apache/spark/pull/20629.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nDeprecate `KMeans.computeCost` which was introduced as a temp fix and now it is not needed anymore, since we introduced `ClusteringEvaluator`.\r\n\r\n## How was this patch tested?\r\n\r\nmanual test (deprecation warning displayed)\r\nScala\r\n```\r\n...\r\nscala> model.computeCost(dataset)\r\nwarning: there was one deprecation warning; re-run with -deprecation for details\r\nres1: Double = 0.0\r\n```\r\n\r\nPython\r\n```\r\n>>> import warnings\r\n>>> warnings.simplefilter('always', DeprecationWarning)\r\n...\r\n>>> model.computeCost(df)\r\n/Users/mgaido/apache/spark/python/pyspark/ml/clustering.py:330: DeprecationWarning: Deprecated in 2.4.0. It will be removed in 3.0.0. Use ClusteringEvaluator instead.\r\n  \" instead.\", DeprecationWarning)\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/383561748","html_url":"https://github.com/apache/spark/pull/20629#issuecomment-383561748","issue_url":"https://api.github.com/repos/apache/spark/issues/20629","id":383561748,"user":{"login":"mgaido91","id":8821783,"avatar_url":"https://avatars1.githubusercontent.com/u/8821783?v=4","gravatar_id":"","url":"https://api.github.com/users/mgaido91","html_url":"https://github.com/mgaido91","followers_url":"https://api.github.com/users/mgaido91/followers","following_url":"https://api.github.com/users/mgaido91/following{/other_user}","gists_url":"https://api.github.com/users/mgaido91/gists{/gist_id}","starred_url":"https://api.github.com/users/mgaido91/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgaido91/subscriptions","organizations_url":"https://api.github.com/users/mgaido91/orgs","repos_url":"https://api.github.com/users/mgaido91/repos","events_url":"https://api.github.com/users/mgaido91/events{/privacy}","received_events_url":"https://api.github.com/users/mgaido91/received_events","type":"User","site_admin":false},"created_at":"2018-04-23T12:44:13Z","updated_at":"2018-04-23T12:44:13Z","author_association":"CONTRIBUTOR","body":"kindly ping @holdenk @MLnick "}}
{"actor":{"display_login":"mgaido91"}}