Skip to content

Implement circular buff#7

Merged
itsmontoya merged 7 commits intomainfrom
implement-circular-buff
Jul 16, 2024
Merged

Implement circular buff#7
itsmontoya merged 7 commits intomainfrom
implement-circular-buff

Conversation

@itsmontoya
Copy link
Copy Markdown
Collaborator

The purpose of this PR is to implement a circular buffer and then utilize that buffer instead of a slice for ngrams.

benchmark                old ns/op     new ns/op     delta
Benchmark_toNGrams-8     708           566           -20.01%

@itsmontoya itsmontoya self-assigned this Jul 16, 2024
Copy link
Copy Markdown

@BrandenWilliams BrandenWilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very exciting!

@itsmontoya itsmontoya merged commit e156b4f into main Jul 16, 2024
@itsmontoya itsmontoya deleted the implement-circular-buff branch July 16, 2024 16:32
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.

2 participants