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

Require in LR LogisticAggregator is redundant

    XMLWordPrintableJSON

Details

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

    Description

      require(numFeatures == features.size, ....)
      require(weight >= 0.0, ...)

      in LogisticAggregator (add and merge) will never be triggered as the dimension and weight has been checked in MultivariateOnlineSummarizer.

      Given the frequent usage of function add, the redundant require should be removed.

      Attachments

        Activity

          People

            wm624 Miao Wang
            yuhaoyan yuhao yang
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: