Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: Benchmark and improve performance. #359

Merged
merged 9 commits into from
May 25, 2021
Merged

feat: Benchmark and improve performance. #359

merged 9 commits into from
May 25, 2021

Conversation

yeldiRium
Copy link
Contributor

@yeldiRium yeldiRium commented May 25, 2021 •
edited
Loading

  • Adds benchmarks for array-diffing, object-diffing and string-diffing.
  • Improves array-diffing performance by comparing arrays linearly, if they are longer than a certain threshold.
  • Improves string-diffing performance by choosing the split character based on string length. If a string is too long it is first chunked by words, then - if still necessary - by lines.

We should wait until strangedev/zufall#8 is merged and then remove the type declarations.

@yeldiRium yeldiRium marked this pull request as ready for review May 25, 2021 15:05
@goloroden goloroden assigned goloroden and unassigned yeldiRium May 25, 2021
@goloroden goloroden enabled auto-merge (squash) May 25, 2021 16:23
@goloroden goloroden merged commit d94891d into main May 25, 2021
@goloroden goloroden deleted the chore/benchmark branch May 25, 2021 16:23
goloroden pushed a commit that referenced this pull request May 25, 2021
@semantic-release-bot
chore(release): 6.3.0 [skip ci]
43f6ff9 
# [6.3.0](6.2.2...6.3.0) (2021-05-25)

### Features

* Benchmark and improve performance. ([#359](#359)) ([d94891d](d94891d))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@goloroden goloroden goloroden approved these changes

@dotKuro dotKuro Awaiting requested review from dotKuro

@strangedev strangedev Awaiting requested review from strangedev

Assignees

@goloroden goloroden

@strangedev strangedev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@yeldiRium @goloroden @strangedev