Commit Graph

136 Commits

Author SHA1 Message Date
beea6b1c35 fix: moveToGroup should rename traces, not merge them
Changed from apiCall('merge_trace', ...) to updateTraceProfileGroup(name=targetGroup.name).
This just changes the trace profile name to join the target group,
without actually merging trace data together.
2026-08-01 05:50:10 +08:00
1c0f15664e fix: batchDeleteGroups should ungroup (rename to Unassigned), not hard delete
Changed from apiCall('delete_trace', ...) to updateTraceProfileGroup(name='Unassigned').
This preserves all trace data and just removes the group assignment.
Traces move back to Pending Faces instead of being deleted.
2026-08-01 05:40:46 +08:00
adc8d944c6 perf: use direct trace-thumb endpoint + fallback for face group thumbnails
- loadTraceThumb now uses /api/v1/trace-thumb/{uuid}/{tid} (~8ms)
- Face group imgs use direct trace-thumb URL with fallback
- Merge dialogs also use direct trace-thumb with fallback
- MAX_CONCURRENT increased to 32

Fallback chain:
1. c.rep_trace.file_uuid/trace_id (full trace data)
2. selectedFileUuid/representative_trace (group data only)
3. SVG silhouette
2026-08-01 05:29:55 +08:00
5be57aec7d Revert "perf: lazy load face group thumbnails + increase concurrency to 32"
This reverts commit 26b94f7243.
2026-08-01 01:58:01 +08:00
26b94f7243 perf: lazy load face group thumbnails + increase concurrency to 32
- Add v-observe directive to face groups for lazy thumbnail loading
- Increase MAX_CONCURRENT from 16 to 32 for faster parallel loading
- Apply lazy loading to merge dialogs and candidate groups
- Only load thumbnails when cards become visible (IntersectionObserver)

Improves initial load time for Charade (153 groups, 2451 traces) from
~76s to <5s for visible groups.
2026-08-01 01:49:45 +08:00
9d58fdb9d9 fix: unregister timeout 200s + async mode
- Add UNREGISTER_TIMEOUT=200000 (200s) for large file unregister
- Add timeout field to buildHttpRequest return type
- httpCall now uses per-request timeout override
- unregister_file now uses asyncMode: true for better UX
- Show "Removing in background..." toast for async mode

Fixes timeout error when unregistering large files (83K faces, 20K nodes).
2026-07-31 18:32:50 +08:00
2b944cf9d1 fix: use search/people API (Core Team deployed)
- Remove duplicate search_people case
- Remove search_identities (no longer needed)
- Add search_people response handler
- SearchView already uses search_people
2026-07-25 23:16:59 +08:00
453168f326 chore: remove unused .gs-sidebar-lock CSS 2026-07-25 21:13:22 +08:00
bcffe62a15 fix: remove lock icon from sidebar toggle 2026-07-25 21:13:07 +08:00
ee9a602f7f fix: toggle shows expand chevron when locked, lock icon when unlocked 2026-07-25 20:57:58 +08:00
3aef7d115f fix: sidebar default to collapsed 2026-07-25 20:56:29 +08:00
19846a8773 revert: restore localStorage-based sidebar default 2026-07-25 20:51:48 +08:00
bb4ff9b007 fix: sidebar default to collapsed (locked) 2026-07-25 20:51:38 +08:00
fcb612ff0e revert: restore search_identities API case 2026-07-25 20:38:54 +08:00
1a0b3f2c1f fix: use search/people API (no identity concept) 2026-07-25 20:38:30 +08:00
9386a8e8ef fix: fallback to search_identities (search/people endpoint not deployed) 2026-07-25 20:35:47 +08:00
fb9a55a64c feat: use new search/people API for People search mode 2026-07-25 20:33:15 +08:00
9691e75e36 fix: use height:100% on sidebar/main (resolves to parent content height) 2026-07-25 20:29:35 +08:00
674126d687 fix: add html/body height:100% + 100% height on sidebar/main
- html,body { height:100% } to ensure proper height chain
- ms-side { height: 100% } and ms-main { height: 100% }
- Both resolve to same parent content height: 100vh - 56px
2026-07-25 20:29:26 +08:00
d37967d38e fix: explicit calc(100vh - 56px) on both sidebar and main 2026-07-25 20:28:56 +08:00
a289346fdc fix: revert to flexbox with height:100vh, fix footer visibility
- height:100vh (not min-height) so align-items:stretch actually works
- Footer shows when expanded, hides when collapsed (dynamic class)
- Remove align-self:stretch (unnecessary with flex stretch)
- Restore flex-shrink on divider
2026-07-25 20:27:09 +08:00
dd843c29ee fix: use CSS grid for equal-height sidebar and main 2026-07-25 20:24:11 +08:00
d7592c528c fix: sync sidebar height with main content area 2026-07-25 20:21:17 +08:00
f75eb44351 feat: show compact login/logout icon when sidebar collapsed 2026-07-25 20:20:03 +08:00
652ba77c77 fix: use CSS instead of v-show for sidebar collapse text
- Replace v-show with CSS .gs-nav-label + .collapsed display: none
- Use CSS class .gs-hidden instead of v-show for footer/divider
- Fix content not re-displaying after collapse/expand
2026-07-25 20:18:03 +08:00
ca94259fba feat: add sidebar collapse/expand toggle 2026-07-25 20:11:56 +08:00
660f01c6c5 fix: People search mode label 2026-07-25 19:27:56 +08:00
cb7ebfb633 feat: add Face Group search mode back to SearchView
- Add 'Face Group' mode to search modes
- Use search_identities API temporarily
- Update summary: 'Identity' → 'FS'
- Will replace with Core Team's search_tkg_node_face_trace_profile API
2026-07-25 19:24:54 +08:00
25f718c0d5 fix: remove duplicate case clauses in api/index.ts 2026-07-25 18:28:04 +08:00
0b19a0af0d refactor: QC report - 'Trace Profile' → 'FS Profile' 2026-07-25 15:53:37 +08:00
cf397b268e refactor: QC - use FS instead of Trace
- Display format: FS#123 instead of Trace #123
- Update labels: 'Traces' → 'FS'
- Update messages: 'clusters' → 'groups'
2026-07-25 15:53:17 +08:00
aa25600580 refactor: remove identity/pending concepts
- identity_uuid → group_uuid
- Remove status: 'pending'
- Remove starred: false
- Keep only: face, FS, face group
2026-07-25 15:51:38 +08:00
464ddd16a9 chore: remove debug console.log from faceGroups 2026-07-25 15:49:27 +08:00
4463ff2f96 style: change 'Faces' → 'FS' in Face Groups card 2026-07-25 15:48:13 +08:00
15647a47ac fix: replace all clusterAsPending → faceGroups 2026-07-25 15:43:28 +08:00
0a7902c2f5 refactor: rename clusterAsPending → faceGroups (no pending concept)
- Rename computed property to match current API semantics
- face_groups are definitive, not pending
- Add detailed console.log for debugging empty groups issue
2026-07-25 15:43:15 +08:00
d3cd940818 fix: revert to clusterAsPending (remove allGroupsForMove) 2026-07-25 15:42:31 +08:00
7cc4708c66 debug: add more console.log for clusterAsPending diagnosis 2026-07-25 15:42:17 +08:00
5db1b68fa0 debug: add allGroupsForMove computed for Move Faces Modal
- Explicitly define data source for Move Faces Modal
- Use clusterAsPending directly
- Add console.log for debugging

Purpose: Diagnose why only one group shows in Move Faces Modal
2026-07-25 15:40:48 +08:00
ec8d8d9d1a fix: add fallback for missing group names in Move Faces Modal
Problem:
- Some groups may not have name field
- Display would show blank or undefined

Solution:
- Add fallback: c.name || 'Unnamed Group'
- Add fallback: c.trace_count || 0
- Ensures always shows meaningful text

Impact:
- Move Faces Modal now always shows group names
- No blank entries even with incomplete data
2026-07-25 15:40:01 +08:00
f727b21e99 refactor: remove unused updateClusterAfterMerge function
- No longer needed with new merge_groups API
- merge_groups API handles group merging server-side
- Frontend only needs to refresh data after merge
2026-07-25 14:25:59 +08:00
8af2dc6128 feat: integrate new merge_groups API from Core Team
Replace merge_trace implementation with merge_groups API

Changes:
1. src/api/index.ts:
   - Add merge_groups API definition
   - POST /api/v1/file/:file_uuid/groups/merge
   - Parameters: file_uuid, source_groups[], target_group_name

2. src/views/PeopleView.vue:
   - Rewrite executeMerge() function
   - Use group names instead of trace IDs
   - Single API call for all groups
   - Remove trace-level loop

3. src/views/PeopleView.vue:
   - Delete updateClusterAfterMerge() function
   - No longer needed with new API

Benefits:
- ✅ Atomic operation (single API call)
- ✅ Correct merge behavior (groups, not traces)
- ✅ Better performance (1 call vs N calls)
- ✅ Data consistency guaranteed

API behavior:
- Merge N groups into 1 target group
- All traces from source groups merged to target
- Source groups deleted after merge
- Returns traces_merged count

Core API: Implemented by Core Team (2026-07-25)
Status: Ready for testing
2026-07-25 14:25:35 +08:00
042bb45574 fix: update Pending Faces display to FS# format
- Line 102: Pending Faces name display
- Changed from F# to FS# (Face Sequence)
- Consistent with other Face Trace displays
2026-07-25 13:55:18 +08:00
e6b9b7b4b8 refactor: add # separator to display format
Display format with # separator:
- Frame: F#{number} (e.g., F#333)
- Face Trace: FS#{number} (e.g., FS#233)

Changes:
- Line 105: Frame range display (F#2219–F#2225)
- Line 143: Face Detail Modal (file_uuid:FS#233)
- Line 262: Cluster strip fallback (FS#233)
- Line 264: Cluster strip label (FS#233)

Benefits:
- # separator makes ID clearer
- Consistent format across all displays
- Visual distinction between type and number
2026-07-25 13:55:01 +08:00
98534bebfb refactor: change Face Trace display from 'Face{id}' to 'FS{id}'
Display format clarification:
- Frame: F{number} (e.g., F333)
- Face Trace: FS{number} (e.g., FS233)

Internal data remains: trace_id = 233 (integer)
Display text: FS233 (Face Sequence)

Changes:
- Line 143: Face Detail Modal display
- Line 262: Cluster strip fallback display
- Line 264: Cluster strip label

Reason:
- FS = Face Sequence (clear meaning)
- Avoid confusion with Frame (F)
- Separation of concerns: internal ID vs display text
2026-07-25 13:54:01 +08:00
389b3c8ba3 feat: add BuildVersion component to App.vue template
- Add BuildVersion at root level of template
- Shows build version in top-right corner on all views
2026-07-25 13:36:47 +08:00
9de4910aa0 feat: add BuildVersion component to App.vue
- Import BuildVersion component
- Add to template (top-right corner fixed)
- Visible on all views

Build version format: 20260725133504-gitHash
Shows in top-right corner with monospace font
2026-07-25 13:36:14 +08:00
baa42fa625 feat: add build version display to all views
Add build version indicator to verify code updates:

1. Create BuildVersion.vue component:
   - Displays in top-right corner (fixed position)
   - Format: YYYYMMDDHHMMSS-gitHash
   - Example: 20260725133504-39ce37e

2. Update vite.config.ts:
   - Inject __BUILD_VERSION__ global variable
   - Generated from build time + git commit hash

3. Add to App.vue:
   - Shown on all views automatically
   - Z-index: 9999 (always visible)

4. Add vite-env.d.ts:
   - TypeScript declarations for __BUILD_VERSION__

Purpose:
- User can verify if browser loaded latest code
- Easy to confirm updates are deployed
- No need to check source code

Location: Top-right corner, fixed position
Format: Monospace font, small text, subtle styling
2026-07-25 13:35:41 +08:00
39ce37e69c fix: remove premature updateClusterAfterMerge call in merge loop
Critical bug fix:
- updateClusterAfterMerge() was called after EACH trace merge
- This caused source group to be deleted after first trace
- Remaining traces had no group to merge into

Problem flow:
1. Merge trace #1 from Group B → Group A
2. updateClusterAfterMerge(B, A) deletes Group B
3. Try to merge trace #2 from Group B → Group B is gone!
4. Result: All source groups deleted after first trace

Solution:
- Remove updateClusterAfterMerge() from loop
- Let all traces merge successfully
- Reload cluster data at the end (Line 1813)
- Core API handles the merge, we just refresh UI

Impact:
- Merge now works correctly for all traces in a group
- No premature deletion of source groups
- UI refreshes after all operations complete
2026-07-25 13:29:47 +08:00
7b5650d098 fix: correct updateClusterAfterMerge to trigger computed update
Problem:
- Previous commit tried to splice computed property (clusterAsPending)
- Computed properties cannot be modified directly
- Need to update underlying data (clusterResultsCache)

Solution:
- Remove incorrect clusterAsPending.value.splice()
- Trigger reactive update via clusterResultsCache reassignment
- clusterAsPending computed will auto-update

Technical details:
- Vue computed properties are read-only
- Must modify underlying reactive data
- Reassignment triggers computed recalculation
2026-07-25 13:25:41 +08:00