Manual merge/split of derived weather events (deferred from #136) #167
Labels
No labels
area:ai
area:ci-cd
area:notifications
area:observability
area:public-pages
backlog
bug
duplicate
enhancement
help wanted
invalid
question
type:decision
type:feature
type:infra
type:maintenance
type:security
v1.0.1
v1.1.0
v1.2.0
v1.3.0
v2.0.0
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
rbrooks/WeatherBot#167
Loading…
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?
Deferred from #136 (decision recorded there, 2026-08-01): v2.0.0 ships the automatic correlation rules only; user-driven merge/split of derived events is explicitly out of scope until real usage shows where the automatic grouping misfires.
Why deferred
A user-override layer that survives rebuilds contradicts the "derived and rebuildable, never source of truth" constraint (#22, #135) and roughly doubles the correlation design: override storage keyed to something stable across re-derivation, conflict semantics when a rebuild produces groupings that no longer match an override, and merge/split UI.
What it would take (sketch, for when this is picked up)
derivation_versionbump changes the base grouping.Prerequisite evidence
The event view already shows why members grouped (gap distances, upgrade edges — per #136). Collect actual misgrouping examples through a convective season before designing this; the gap constant (
EVENT_GAP_MINUTES=90) can also simply be retuned via rebuild, which may make many desired overrides unnecessary.