Add/expose random seed in LearningPipeline to get deterministic results #9

Activity

1. markusweimer commented on May 4, 2018

   

   markusweimer

   on May 4, 2018

   More actions

   Which of those variants do you prefer?

     // Variant 1: Constructor
     var p = new LearningPipeline(seed:1234);
   
     // Variant 2: Property
     p.seed = 1234;
   
     // Variant 3: Parameter to .Train()
     var m = p.Train(seed:1234);
2. shauheen

   added

   enhancementNew feature or requestNew feature or request

   up-for-grabs

   on May 9, 2018
3. shauheen

   added this to the 0518 milestone on May 9, 2018
4. shauheen

   removed

   up-for-grabs

   on May 9, 2018
5. zeahmed commented on May 11, 2018

   

   zeahmed

   on May 11, 2018

   Last edited by zeahmed

   Contributor

   More actions

   I'll prefer

   // Variant 3: Parameter to .Train()
   var m = p.Train(seed:1234);

   👍React with 👍1Reacted by Remberto Martinez
6. Ivanidzo4ka

   closed this as completedon May 11, 2018
7. Ivanidzo4ka

   reopened this on May 11, 2018
8. TomFinley commented on May 11, 2018

   

   TomFinley

   on May 11, 2018

   Last edited by TomFinley

   Contributor

   More actions

   Seriously, enough is enough I think. Let's just expose IHostEnvironment. That's our mechanism to solve this and other problems, it's worked well for years doing exactly this among all its other functions.

   👍React with 👍1Reacted by Gleb K
9. shauheen

   modified the milestones: 0518, 0618 on May 22, 2018
10. shauheen commented on Jun 28, 2018

    

    shauheen

    on Jun 28, 2018

    Contributor

    More actions

    With a few PRs already merged as well as the proposed #371 , this issue can now be closed.
11. shauheen

    closed this as completedon Jun 28, 2018
12. ghost locked as resolved and limited conversation to collaborators on Mar 31, 2022