Update models for v2 personalization
Categories
(Firefox :: New Tab Page, task, P1)
Tracking
()
People
(Reporter: thecount, Assigned: thecount)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
A new list of models incoming
Roughly this:
"nb_model_arts_and_entertainment, nb_model_autos_and_vehicles, nb_model_beauty_and_fitness, nb_model_blogging_resources_and_services, nb_model_books_and_literature, nb_model_business_and_industrial, nb_model_computers_and_electronics, nb_model_finance, nb_model_food_and_drink, nb_model_games, nb_model_health, nb_model_hobbies_and_leisure, nb_model_home_and_garden, nb_model_internet_and_telecom, nb_model_jobs_and_education, nb_model_law_and_government, nb_model_online_communities, nb_model_people_and_society, nb_model_pets_and_animals, nb_model_real_estate, nb_model_reference, nb_model_science, nb_model_shopping, nb_model_sports, nb_model_travel"
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Updated•5 years ago
|
Comment 3•5 years ago
|
||
bugherder |
Assignee | ||
Comment 4•5 years ago
|
||
[Tracking Requested - why for this release]:
Assignee | ||
Comment 5•5 years ago
|
||
Comment on attachment 9126834 [details]
Bug 1615734 - Update v2 models
Beta/Release Uplift Approval Request
- User impact if declined: Not much.
It makes experiment setup easier, which in turn causes our experiment to run smoother, less chance for something to break, and less chance for a problematic experience for the users in the experiment.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce: Not really needed, the experience with this change should be exactly the same as without, so maybe some basic regression testing would be nice, but I'm pretty confident it'll be fine based on how simple the changes are.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It's just updating the default value in a pref that's used in an experiment. (it's also the value we use in nightly)
- String changes made/needed: none
Comment 6•5 years ago
|
||
Comment on attachment 9126834 [details]
Bug 1615734 - Update v2 models
Low risk, uplift approved for 74.0b6, thanks.
Comment 7•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
Description
•