Skip to content
This repository was archived by the owner on Oct 18, 2022. It is now read-only.

Faster WaveletTreeNoptrs::rank(): save up to log(sigma) binary rank rank operations.#4

Merged
fclaude merged 1 commit intofclaude:masterfrom
mpetri:ranknoptrfix
Aug 17, 2011
Merged

Faster WaveletTreeNoptrs::rank(): save up to log(sigma) binary rank rank operations.#4
fclaude merged 1 commit intofclaude:masterfrom
mpetri:ranknoptrfix

Conversation

@mpetri
Copy link
Contributor

@mpetri mpetri commented Aug 17, 2011

In WaveletTreeNoptrs::rank() bitstring[level]->rank1(start-1) is calculated twice. Storing it is more efficient.

@fclaude
Copy link
Owner

fclaude commented Aug 17, 2011

WOW! I thought I had changed that after testing for correctness of the
structure ... that's pretty bad! Thanks a lot for the report!

Cheers,
Francisco.

On Tue, Aug 16, 2011 at 8:50 PM, mpetri
reply@reply.github.com
wrote:

In WaveletTreeNoptrs::rank() bitstring[level]->rank1(start-1) is calculated twice. Storing it is more efficient.

Reply to this email directly or view it on GitHub:
#4

fclaude added a commit that referenced this pull request Aug 17, 2011
Faster WaveletTreeNoptrs::rank(): save up to log(sigma) binary rank rank operations.
@fclaude fclaude merged commit 60835b4 into fclaude:master Aug 17, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments