Mahout
  1. Mahout
  2. MAHOUT-812

Allow ConfusionMatrix to be Writable (via MatrixWritable)

    Details

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

      Description

      ConfusionMatrix does not support Writable. This patch adds that feature. ConfusionMatrix is a subclass of MatrixWritable.

      Since ConfusionMatrix is somewhat less than useful without the row/column labels, and MatrixWritable does not support writing bindings (it only saves numbers), this patch fixes both.

      Includes unit test for ConfusionMatrix (previously missing) which includes exercise of MatrixWritable support for numbers and labels. (There is no independent unit test for MatrixWritable.)

      1. MAHOUT-812.patch
        14 kB
        Sean Owen
      2. MAHOUT-812.patch
        16 kB
        Lance Norskog
      3. MAHOUT-812.patch
        13 kB
        Lance Norskog

        Issue Links

          Activity

          Lance Norskog created issue -
          Lance Norskog made changes -
          Field Original Value New Value
          Description ConfusionMatrix does not support Writable. This patch adds that feature.

          Since ConfusionMatrix is somewhat less than useful without the row/column labels, and MatrixWritable does not support writing bindings (it only saves numbers), this patch fixes both.

          Includes unit test for ConfusionMatrix (previously missing) and enhanced unit test for MatrixWritable.
          ConfusionMatrix does not support Writable. This patch adds that feature. ConfusionMatrix is a subclass of MatrixWritable.

          Since ConfusionMatrix is somewhat less than useful without the row/column labels, and MatrixWritable does not support writing bindings (it only saves numbers), this patch fixes both.

          Includes unit test for ConfusionMatrix (previously missing) which includes exercise of MatrixWritable support for numbers and labels. (There is no independent unit test for MatrixWritable.)
          Lance Norskog made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Lance Norskog made changes -
          Attachment MAHOUT-812.patch [ 12494954 ]
          Lance Norskog made changes -
          Attachment MAHOUT-812.patch [ 12497390 ]
          Lance Norskog made changes -
          Summary Enhance ConfusionMatrix to be a MatrixWritable (enhances MatrixWritable to pickle labels) Allow ConfusionMatrix to be Writable (via MatrixWritable)
          Sean Owen made changes -
          Attachment MAHOUT-812.patch [ 12497395 ]
          Sean Owen made changes -
          Assignee Sean Owen [ srowen ]
          Sean Owen made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Joe Prasanna Kumar made changes -
          Link This issue is depended upon by MAHOUT-838 [ MAHOUT-838 ]
          Sean Owen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Sean Owen
              Reporter:
              Lance Norskog
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development