Skip to content

Hyperparameters reversed in Scenario3  #25

Closed

Description

@maryamariyan

From @justinormont on May 2, 2018, 8:51 AM PDT

Line below on file machinelearning/test/Microsoft.ML.Tests/Scenarios/Scenario3_SentimentPrediction.cs

CharFeatureExtractor = new NGramNgramExtractor() { NgramLength = 2, AllLengths = true },

Currently: (trigrams & unichargrams+bichargrams)

  CharFeatureExtractor = new NGramNgramExtractor() { NgramLength = 2, AllLengths = true },
  WordFeatureExtractor = new NGramNgramExtractor() { NgramLength = 3, AllLengths = false }

Should be: (unigram+bigram & trichargrams)

  CharFeatureExtractor = new NGramNgramExtractor() { NgramLength = 3, AllLengths = false },
  WordFeatureExtractor = new NGramNgramExtractor() { NgramLength = 2, AllLengths = true }

Activity

  1. added this to the 0518 milestone on May 8, 2018
  2. added a commit that references this issue on May 10, 2018
    ff5fb14
  3. added a commit that references this issue on Jul 27, 2018
    7ba8617
  4. added a commit that references this issue on Aug 22, 2019
  5. ghost locked as resolved and limited conversation to collaborators on Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions