Repository navigation
[Meta Issue] Changing defaults #4749
Description
Activity
Assigning @gvashishtha for triage.
@justinormont correct me if I am wrong: was the default weighting TF-IDF in TLC? It is currently TF
The default text recipe in TLC uses TF. I believe the current default in ML․NET is also TF, so no changes needed for that.
Reacted by Najeeb KazmiChanging a default parameter value is an acceptable change. Just be sure you still support the old default value - for example if the old default was ‘null’ and you change it to a non-null default, you still should support ‘null’ values.
- ghost locked as resolved and limited conversation to collaborators
on Mar 19, 2022
Metadata
Metadata
Assignees
Labels
No labels
First of all, is this a breaking change? @terrajobst suggested it is an acceptable breaking change #2305 (comment), @eerhardt commented that is was decided that changing defaults is acceptable #2305 (comment)
Change the following defaults based on results from @justinormont:
Update default n-gram length for Text Transform to match default text recipe #2870
FeatirizeTextdefault n-grams lengths to match default text recipe:Change default # of iterations in Averaged Perceptron to 10 #2305
AveragedPerceptrondefault iterations from 1 to 10RowGroupColumnName of ranking trainers options class defaults to null #3365
RowGroupColumnNameinFastTreeandLightGbmranking trainerOptionsclasses:null->'GroupId'Optionsclass, where the default is'GroupId'nulleverywhere or'GroupId'everywhere.cc: @harishsk @justinormont @gvashishtha @eerhardt