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

[FEATURE] Added new method betweenLexoRanks which allows to generate multiple between lexoRank based on user input #1

Conversation

mananrshah
Copy link

@mananrshah mananrshah commented Oct 16, 2024

Closes issue reported on upstream repo: kvandake#28.

A new method betweenLexoRanks has been introduced to efficiently generate the intermediate lexoRank values in bulk. Previously, the process involved identifying the exact middle rank each time, leading to rapid growth of lexoRanks. The updated approach involves creating a slice based on the desired number of lexoRanks, optimizing the computation by using slices instead of repeatedly halving.

@oliverlockwood oliverlockwood force-pushed the master branch 8 times, most recently from ebc6cfb to 3946dac Compare October 18, 2024 09:02
@oliverlockwood oliverlockwood changed the title Added new method betweenLexoRanks which allows to generate multiple between lexoRank based on user input [FEATURE] Added new method betweenLexoRanks which allows to generate multiple between lexoRank based on user input Oct 18, 2024
@oliverlockwood oliverlockwood merged commit c16ac45 into dalet-oss:master Oct 18, 2024
2 checks passed
@oliverlockwood oliverlockwood deleted the support_to_generate_multiple_in_between_lexorank branch October 18, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants