{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/14653","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/14653/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/14653/comments","events_url":"https://api.github.com/repos/apache/spark/issues/14653/events","html_url":"https://github.com/apache/spark/pull/14653","id":171272037,"number":14653,"title":"[SPARK-10931][PYSPARK][ML] PySpark ML Models should contain Param values","user":{"login":"evanyc15","id":4530799,"avatar_url":"https://avatars.githubusercontent.com/u/4530799?v=3","gravatar_id":"","url":"https://api.github.com/users/evanyc15","html_url":"https://github.com/evanyc15","followers_url":"https://api.github.com/users/evanyc15/followers","following_url":"https://api.github.com/users/evanyc15/following{/other_user}","gists_url":"https://api.github.com/users/evanyc15/gists{/gist_id}","starred_url":"https://api.github.com/users/evanyc15/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/evanyc15/subscriptions","organizations_url":"https://api.github.com/users/evanyc15/orgs","repos_url":"https://api.github.com/users/evanyc15/repos","events_url":"https://api.github.com/users/evanyc15/events{/privacy}","received_events_url":"https://api.github.com/users/evanyc15/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":19,"created_at":"2016-08-15T21:59:19Z","updated_at":"2016-10-25T21:15:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/14653","html_url":"https://github.com/apache/spark/pull/14653","diff_url":"https://github.com/apache/spark/pull/14653.diff","patch_url":"https://github.com/apache/spark/pull/14653.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nChanged PySpark models to include the Param values.\r\nRefer to the closed PR #10270  for additional information.\r\n\r\n## How was this patch tested?\r\n\r\nTested using Python doctests\r\n\r\n## Changesets:\r\n\r\n\r\nEstimator UID is being copied correctly to the Transformer model objects and params now, working on Doctests\r\n\r\nChanged the way parameters are copied from the Estimator to Transformer\r\n\r\nCheckpoint, switching back to inheritance method\r\n\r\nWorking on DocTests\r\n\r\nImplemented Doctests for Recommendation, Clustering, Classification (except RandomForestClassifier), Evaluation, Tuning, Regression (except RandomRegression)\r\n\r\nReady for Code Review\r\n\r\nCode Review changeset #1"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/256178211","html_url":"https://github.com/apache/spark/pull/14653#issuecomment-256178211","issue_url":"https://api.github.com/repos/apache/spark/issues/14653","id":256178211,"user":{"login":"jkbradley","id":5084283,"avatar_url":"https://avatars.githubusercontent.com/u/5084283?v=3","gravatar_id":"","url":"https://api.github.com/users/jkbradley","html_url":"https://github.com/jkbradley","followers_url":"https://api.github.com/users/jkbradley/followers","following_url":"https://api.github.com/users/jkbradley/following{/other_user}","gists_url":"https://api.github.com/users/jkbradley/gists{/gist_id}","starred_url":"https://api.github.com/users/jkbradley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jkbradley/subscriptions","organizations_url":"https://api.github.com/users/jkbradley/orgs","repos_url":"https://api.github.com/users/jkbradley/repos","events_url":"https://api.github.com/users/jkbradley/events{/privacy}","received_events_url":"https://api.github.com/users/jkbradley/received_events","type":"User","site_admin":false},"created_at":"2016-10-25T21:15:35Z","updated_at":"2016-10-25T21:15:35Z","body":"@evanyc15 OK back for real now...sorry for the delay.  @BryanCutler has a lot of good comments.  Could you please address them?  Regarding splitting this up into multiple PRs, I strongly +1 that in general, though I'm OK if you want to do this as a batch.\r\n\r\nI'll test this PR out now..."}}
{"actor":{"display_login":"jkbradley"}}