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

Phonet performance (AU and ZA) #33

Open
aarondandy opened this issue Jul 1, 2017 · 1 comment
Open

Phonet performance (AU and ZA) #33

aarondandy opened this issue Jul 1, 2017 · 1 comment

Comments

@aarondandy
Copy link
Owner

The method WordList.QuerySuggest.Phonet has two nested loops that query for entries from a PhoneTable that have rules starting with a given character. Indexing these entries in the table by the first rule character may have a positive impact on performance.

@aarondandy
Copy link
Owner Author

aarondandy commented Jul 2, 2017

This may only have an impact on Australian and South African English dictionaries. A specialized performance benchmark may need to be added to notice a difference or just be lazy and modify the performance tests to use an Australian dictionary instead of an American one.

@aarondandy aarondandy changed the title Phonet performance Phonet performance (AU and AZ) Jul 2, 2017
@aarondandy aarondandy changed the title Phonet performance (AU and AZ) Phonet performance (AU and ZA) Jul 2, 2017
@aarondandy aarondandy added this to the 4.0.0 milestone Feb 23, 2022
@aarondandy aarondandy removed this from the 4.0 milestone May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant