Sort out the pool timeline UI widget. #193
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Right now it's barely workable, and the key takes up way too much data.
Ensure that the timeline updates when the user manually sets the pool condition.
Hi diddly-ho Eli — dug into the pool timeline widget (
ts/components/RevealTimeline.vue, used in the Reveal workbench detail panel) to map out what "barely workable" and "key takes up too much data" mean in the code, plus what it takes to make the timeline track manual condition sets.Where the widget stands today
feature_pool_stateviaGET /api/feature/{id}→pool.state_history, loaded inReveal.vue:loadFeatureHistory()when a pool is selected. Export markers come frompool.exports(currently always[]from real data).pool.detectionsonly, Export chip only when exports exist) is actually already written on theissue-176-n-itemsbranch (N7, commit3dbc4139) but that PR (#188) is stalled/not mergeable, so it never landed on main.activeYeardefaults to the current calendar year, not the latest year that has detections — a pool whose history is in 2024–2025 renders an empty track until you click a year tab.imageryLoaded = trueand shows a banner; it doesn't load anything.On the timeline updating when the condition is manually set
Today the manual condition set happens in the review interfaces (
sudo/RevealReview.vue/review/VisionTaskReview.vue→putVisionTask→ reviewed condition saved; on analysis commit,platform/vision_analysis_commit.goinserts a newfeature_pool_staterow — which is the timeline data). The gap:featureStore.fetchByIDcaches by ID and never invalidates, so the Reveal workbench timeline keeps showing the stale history until a full page reload. That's the concrete thing to fix for your requirement.Proposed plan (small PR on main)
featureStore(or invalidate on select) soloadFeatureHistoryrefetchesstate_historyafter a manual condition set + commit, instead of serving the cached detail.One question to scope #4 correctly: should the timeline also be embedded in the review UI (sudo RevealReview / VisionTaskReview) so the reviewer sees the pool's history while setting the condition and it updates live? Or is refreshing the workbench timeline after commit the intended behavior? I can wire either.