c/ietf-draft-analyzer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(webui): timeline embedding landscape — sort points chronologically

Browse Source 
The animated t-SNE landscape delivered points in embedding-dict order, not
chronological order. The front-end cumulative filter (p.month <= frame) then
inserted new points mid-array between frames, so Plotly's index-based frame
transition animated existing markers flying to other drafts' coordinates.
Visible symptom: a couple of points jumping around instead of a growing map.

Sorting points by (month, name) makes each frame's per-category marker list
an append-only prefix of the next, so transitions only add markers.
Verified on the live DB: 188 non-append-only frame transitions before, 0 after.
This commit is contained in:
Christian Nennemann
2026-05-22 11:41:02 +02:00
parent 6538cbebaf
commit c3af38e0f9
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/webui/data/analysis.py
View File

@@ -559,6 +559,12 @@ def _compute_timeline_animation_data(db: Database) -> dict:
"month": month, "month": month,
}) })
# Deliver points in chronological order so the front-end's cumulative
# filter (p.month <= frame) is append-only. Otherwise new points get
# inserted mid-array and Plotly's index-based frame transition animates
# existing markers flying to other drafts' coordinates ("jumping points").
points.sort(key=lambda p: (p["month"], p["name"]))
months = sorted(month_set) months = sorted(month_set)
# Convert defaultdict to plain dict for JSON # Convert defaultdict to plain dict for JSON
cat_monthly_plain = {m: dict(cats) for m, cats in category_monthly.items()} cat_monthly_plain = {m: dict(cats) for m, cats in category_monthly.items()}