Closed
Description
When changing AveragedPerceptron's default number of iterations from 1 to 10 in #5258, we missed one.
The main interface continues to default to 1 iteration:
Previous issues:
- [Meta Issue] Changing defaults #4749
- Updated AveragedPerceptron default iterations from 1 to 10 #5258
Code location
Default is set here:
Which originates from:
Which in-turn inherits its value from:
Possible fix
We may want to make a new class within AveragedPerceptron to hold its overrides:
internal class AveragedPerceptronDefault : AveragedDefault
{
public const float NumberOfIterations = 10;
}
/cc @michaelgsharp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment