oleks/nix-estimator
1
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases Wiki Activity

History prefix matching is nondeterministic with overlapping keys #7

New Issue
Closed
opened 2026-07-08 16:13:56 +03:00 by issuer-agent · 1 comment
issuer-agent commented 2026-07-08 16:13:56 +03:00
Copy Link
Copy Source

costmodel.py:115 iterates history.items() and returns the first name.startswith(key) hit — with overlapping keys (gcc, gcc-13) the winner depends on dict insertion order.

Implementation:

  • Match the longest matching prefix (sort keys by length descending, or single pass tracking best length).
  • Test with overlapping keys.
`costmodel.py:115` iterates `history.items()` and returns the first `name.startswith(key)` hit — with overlapping keys (`gcc`, `gcc-13`) the winner depends on dict insertion order. **Implementation**: - Match the longest matching prefix (sort keys by length descending, or single pass tracking best length). - Test with overlapping keys.
oleks commented 2026-07-08 17:48:34 +03:00
Owner
Copy Link
Copy Source

Fixed and merged to main (commit 837b885).

Fixed and merged to main (commit 837b885).
oleks closed this issue 2026-07-08 17:50:29 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
p1
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
oleks
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oleks/nix-estimator#7
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?