Mahout
  1. Mahout
  2. MAHOUT-73

addPoint, computeCentroid can be optimized by using vector operators

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Clustering
    • Labels:
      None
    1. MAHOUT-73.patch
      2 kB
      Pallavi Palleti

      Issue Links

        Activity

        Pallavi Palleti created issue -
        Pallavi Palleti made changes -
        Field Original Value New Value
        Link This issue is a clone of MAHOUT-68 [ MAHOUT-68 ]
        Pallavi Palleti made changes -
        Summary This is still an improvement over existing implementations. addPoint, computeCentroid can be optimized by using vector operators
        Hide
        Pallavi Palleti added a comment -

        We can use Vector operators in addPoint and computeCentroid methods of Canopy class. There by, this can be optimized for Sparse vector.Also, by using vector operators, we are hiding the implementation details and reusing the existing code in Sparse Vector, Dense Vector classes.

        Show
        Pallavi Palleti added a comment - We can use Vector operators in addPoint and computeCentroid methods of Canopy class. There by, this can be optimized for Sparse vector.Also, by using vector operators, we are hiding the implementation details and reusing the existing code in Sparse Vector, Dense Vector classes.
        Pallavi Palleti made changes -
        Attachment MAHOUT-73.patch [ 12387699 ]
        Hide
        Ted Dunning added a comment -


        Why was mahout-68 cloned to produce this issue? It seems like the same thing.

        Show
        Ted Dunning added a comment - Why was mahout-68 cloned to produce this issue? It seems like the same thing.
        Hide
        Pallavi Palleti added a comment -

        I have cloned this inorder to start this issue afresh.

        Show
        Pallavi Palleti added a comment - I have cloned this inorder to start this issue afresh.
        Allen Day made changes -
        Link This issue is part of MAHOUT-77 [ MAHOUT-77 ]
        Hide
        Allen Day added a comment -

        I also addressed this in MAHOUT-77.

        Show
        Allen Day added a comment - I also addressed this in MAHOUT-77 .
        Grant Ingersoll made changes -
        Workflow jira [ 12436884 ] no-reopen-closed, patch-avail [ 12444656 ]
        Grant Ingersoll made changes -
        Assignee Grant Ingersoll [ gsingers ]
        Hide
        Grant Ingersoll added a comment -

        Will fix in MAHOUT-139

        Show
        Grant Ingersoll added a comment - Will fix in MAHOUT-139
        Grant Ingersoll made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Sean Owen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Grant Ingersoll
            Reporter:
            Pallavi Palleti
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development