Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-13785

Deprecate model field in ML model summary classes

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 2.0.0
    • ML
    • None

    Description

      ML model summary classes (e.g., LinearRegressionSummary) currently expose a field "model" containing the parent model. It's weird to have this circular reference, and I don't see a good reason why the summary should expose it (unless I'm forgetting some decision we made before...).

      I'd propose to deprecate that field in 2.0 and to remove it in 2.1.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            josephkb Joseph K. Bradley
            josephkb Joseph K. Bradley
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Issue deployment