Skip to content

Optimize LogField performance - #585

Merged
jwbonner merged 1 commit into
mainfrom
optimize-log-field
Nov 12, 2026
Merged

jwbonner merged 1 commit into
mainfrom
optimize-log-field

Conversation

@jwbonner

Copy link
Copy Markdown
Member

Improves the performance of LogField and related indexing of time-series data. This impacts many operations that interact with log data, but also addresses a critical performance issue when parsing WPILOG data that includes many duplicate records (same field and timestamp). This often occurs with owlet-exported logs or other inefficiently generated WPILOG files.

Key changes:

  • Added binary search helpers with fast paths for common cases (used in place of linear searches across the app)
  • Improved logic for data insertion in LogField (reduces allocations and improves fast path logic in several places)
  • Comparison of raw values checks bytes directly instead of allocating temporary arrays

@jwbonner
jwbonner merged commit 646e7ee into main Nov 12, 2026
28 checks passed
@jwbonner
jwbonner deleted the optimize-log-field branch November 12, 2026 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant