-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add/expose random seed in LearningPipeline to get deterministic results #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
enhancement
New feature or request
Milestone
Comments
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); |
I'll prefer // Variant 3: Parameter to .Train()
var m = p.Train(seed:1234); |
Seriously, enough is enough I think. Let's just expose |
This was referenced May 29, 2018
With a few PRs already merged as well as the proposed #371 , this issue can now be closed. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
It is important to be able to set a random seed for ML experiments so the results are reproducible. Add random seed to LearningPipeline (or elsewhere) to ensure a full experiment is deterministic.
The text was updated successfully, but these errors were encountered: