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.

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

Optimised softOperator and added Trie implementation #159

Merged
merged 6 commits into from
Jan 30, 2023

Conversation

j-mie6
Copy link
Owner

@j-mie6 j-mie6 commented Jan 30, 2023

Improved softOperator, similar to the improvements for softKeyword. In the process a Trie has been added, that will be optimised further moving forward (pending benchmarking).

@j-mie6 j-mie6 added the patch This change doesn't affect the API label Jan 30, 2023
@codeclimate
Copy link

codeclimate bot commented Jan 30, 2023

Code Climate has analyzed commit 5e07469 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 96.1% (50% is the threshold).

This pull request will bring the total coverage in the repository to 90.0% (0.0% change).

View more on Code Climate.

Repository owner deleted a comment from codeclimate bot Jan 30, 2023
@j-mie6 j-mie6 merged commit 8d9d48d into master Jan 30, 2023
@j-mie6 j-mie6 deleted the patch/soft-operator+trie branch January 30, 2023 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch This change doesn't affect the API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant