Skip to content

Refactor Scenario tests... #32

Closed
Closed

Description

@zeahmed

Issue: In Scenarios, there is a class named "Top5Scenarios"
Depending on how you count, there are either 2 or 4 scenarios.
It is recommended renaming the class.


Let's also review the names of the files and make sure they are descriptive.

Activity

  1. changed the title [-]Refectore Scenario tests...[/-] [+]Refactor Scenario tests...[/+] on May 7, 2018
  2. added a commit that references this issue on May 8, 2018
  3. glebuk commented on May 8, 2018

    @glebuk
    Contributor

    PR 52 closed this
    #52

  4. m-a-i-n-g-i commented on May 8, 2018

    @m-a-i-n-g-i

    @glebuk
    Just a quick heads up letting you know that landing this PR #52 just broke one of the links on the main README.md

    • Go to the Examples section and try following the link sending you to the complete sample. Here's the markdown snippet.
    ## Examples
    
    Here's an example of code to train a model to predict sentiment from text samples. (You can see the complete sample [here](https://github.com/dotnet/machinelearning/blob/master/test/Microsoft.ML.Tests/Scenarios/Scenario3_SentimentPrediction.cs)):
    

    Cheers!

  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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions