Mahout
  1. Mahout
  2. MAHOUT-861

Simultaneously update left and right vectors while training ExpectationMaximizationSVDFactorizer

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.6
    • Fix Version/s: 0.6
    • Labels:
      None

      Description

      I guess the updates on left and right vectors in ExpectationMaximizationSVDFactorizer#train(int i, int j, int f, SVDPreference pref) should be done simultaneously according to the original algorithm (http://sifter.org/~simon/journal/20061211.html)

      1. MAHOUT-861.patch
        1.0 kB
        Gokhan Capan

        Activity

        Hide
        Hudson added a comment -

        Integrated in Mahout-Quality #1135 (See https://builds.apache.org/job/Mahout-Quality/1135/)
        MAHOUT-861 Simultaneously update left and right vectors while training ExpectationMaximizationSVDFactorizer

        ssc : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1196517
        Files :

        • /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/recommender/svd/ExpectationMaximizationSVDFactorizer.java
        Show
        Hudson added a comment - Integrated in Mahout-Quality #1135 (See https://builds.apache.org/job/Mahout-Quality/1135/ ) MAHOUT-861 Simultaneously update left and right vectors while training ExpectationMaximizationSVDFactorizer ssc : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1196517 Files : /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/recommender/svd/ExpectationMaximizationSVDFactorizer.java
        Hide
        Sebastian Schelter added a comment -

        Nice spot, Gokhan, thank you very much.

        Show
        Sebastian Schelter added a comment - Nice spot, Gokhan, thank you very much.
        Hide
        Gokhan Capan added a comment -

        Here is the patch of two-lines fix

        Show
        Gokhan Capan added a comment - Here is the patch of two-lines fix

          People

          • Assignee:
            Sebastian Schelter
            Reporter:
            Gokhan Capan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development