[Models] Improve Clustering for ATG
Categories
(Core :: Machine Learning: Frontend, task)
Tracking
()
People
(Reporter: vbaungally, Assigned: vbaungally)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [aiaug])
Attachments
(2 files, 1 obsolete file)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
Need to think about improvements (e.g. using the LLM possibly for grouping)
Updated•23 days ago
|
| Assignee | ||
Updated•23 days ago
|
| Assignee | ||
Comment 1•17 days ago
|
||
This patch adds the missed tab title preprocessin that was present in Smart Tab Grouping.
It also reduces false positives in clusters so groups have higher cohesion.
Comment 3•15 days ago
|
||
| bugherder | ||
Comment 4•15 days ago
|
||
This is a hard thing to verify but from what I noticed, even with tabs from the same domain, Organize tabs will show separate Tab groups more specific to those pages. Verified as fixed in our latest Nightly 156.0a1 (2026-08-19)
| Assignee | ||
Comment 5•15 days ago
|
||
This patch adds the missed tab title preprocessing that was present in Smart Tab Grouping.
It also reduces false positives in clusters so groups have higher cohesion.
Original Revision: https://phabricator.services.mozilla.com/D319192
Updated•15 days ago
|
Comment 6•15 days ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: There are two features in Auto Tab Grouping for Smart Window - clustering / creating groups and naming the groups.
We have received feedback from Foxfooding that both could use improvements.
This patch improves the clustering portion of ATG. It creates more topically similar groups and ensures that they are more precise.
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: no
- Steps to reproduce for manual QE testing: Open a couple of tabs that are related (e.g. this bug and its patch) and enable smart window. Click on the "window type" icon that shows up on the top right bar. This should show a group name along with the two related tabs.
- Risk associated with taking this patch: low
- Explanation of risk level: It's just config changes and a small text preprocessing update.
- String changes made/needed?: No
- Is Android affected?: no
Comment 7•15 days ago
|
||
firefox-release Uplift Approval Request
- User impact if declined/Reason for urgency: There are two features in Auto Tab Grouping for Smart Window - clustering / creating groups and naming the groups.
We have received feedback from Foxfooding that both could use improvements.
This patch improves the clustering portion of ATG. It creates more topically similar groups and ensures that they are more precise.
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: no
- Steps to reproduce for manual QE testing: Open a couple of tabs that are related (e.g. this bug and its patch) and enable smart window. Click on the "window type" icon that shows up on the top right bar. This should show a group name along with the two related tabs.
- Risk associated with taking this patch: low
- Explanation of risk level: It's just config changes and a small text preprocessing update.
- String changes made/needed?: No
- Is Android affected?: no
| Assignee | ||
Comment 8•15 days ago
|
||
This patch adds the missed tab title preprocessing that was present in Smart Tab Grouping.
It also reduces false positives in clusters so groups have higher cohesion.
Original Revision: https://phabricator.services.mozilla.com/D319192
Comment 9•15 days ago
|
||
(In reply to Phabricator Automation from comment #7)
firefox-release Uplift Approval Request
- User impact if declined/Reason for urgency: There are two features in Auto Tab Grouping for Smart Window - clustering / creating groups and naming the groups.
We have received feedback from Foxfooding that both could use improvements.
This patch improves the clustering portion of ATG. It creates more topically similar groups and ensures that they are more precise.
This doesn't really answer the "Why now?" aspect of the question. We're rolling out shortened release cycles with one specific goal of cutting down on the number of feature uplifts going into Beta and especially dot releases. This has had basically no Nightly bake time and we're already only a week away from 155 going to RC, so IMO this should ride the 156 release shipping on 15-Sep. If there's a hard product need for it to ship earlier than that, then the case for that needs to be spelled out more clearly.
Updated•15 days ago
|
| Assignee | ||
Comment 10•15 days ago
|
||
There is a product requirement unfortunately. This feature is being tied to 154 dot release for which there is telemetry going on and there's a decision to stop development if the success criteria isn't met. Without this patch and the one that improves naming (https://phabricator.services.mozilla.com/D319836), the success criteria (85% tabs being approved) is unlikely to be met.
If it's too tight a deadline, that's ok. I'll bring this up with the product folks.
Updated•15 days ago
|
| Assignee | ||
Comment 11•15 days ago
|
||
Discussed with product, ok for 155 beta, no release uplift.
Updated•14 days ago
|
Updated•14 days ago
|
Updated•14 days ago
|
Updated•14 days ago
|
Comment 12•14 days ago
|
||
| uplift | ||
Comment 13•14 days ago
|
||
Verified as fixed in our latest Beta 155.0b3
Description
•