[SPARK-5399] tree Losses strings should match loss names

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Duplicate
• Affects Version/s: 1.2.0, 1.2.1
• Component/s: MLlib
• Assignee: Kai
• Reporter: Joseph K. Bradley
• Created: 25/Jan/15 00:21
• Updated: 06/Mar/15 13:02
• Resolved: 06/Mar/15 13:02

Description

tree.loss.Losses.fromString expects certain String names for losses. These do not match the names of the loss classes but should. I believe these strings were the original names of the losses, and we forgot to correct the strings when we renamed the losses.

Currently:

    case "leastSquaresError" => SquaredError
    case "leastAbsoluteError" => AbsoluteError
    case "logLoss" => LogLoss

Proposed:

    case "SquaredError" => SquaredError
    case "AbsoluteError" => AbsoluteError
    case "LogLoss" => LogLoss

Issue Links

• duplicates: SPARK-6113 Stabilize DecisionTree and ensembles APIs
• links to: [Github] Pull Request #4206 (Lewuathe)