Skip to content

Commit

Permalink
Merge pull request #2756 from SGSSGene/fix/large_bi_fm_index
Browse files Browse the repository at this point in the history
[FIX] Constructing large fm_indices
  • Loading branch information
marehr authored Aug 9, 2021
2 parents 0e28954 + c9a515f commit 4604f1b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,15 @@ The following API changes should be documented as such:
If possible, provide tooling that performs the changes, e.g. a shell-script.
-->

# 3.1.0

## Notable Bug-fixes

#### Search

* Resolved an issue that prevented the FM-Index from being constructed correctly for inputs larger than 4 GiB
([\#2756](https://github.com/seqan/seqan3/pull/2756)).

# 3.0.3

Note that 3.1.0 will be the first API stable release and interfaces in this release might still change.
Expand Down
2 changes: 1 addition & 1 deletion include/seqan3/search/fm_index/fm_index.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ class fm_index
size_t const number_of_texts{text_sizes.size()};

// text size including delimiters
size_t const text_size = std::accumulate(text_sizes.begin(), text_sizes.end(), 0) + number_of_texts;
size_t const text_size = std::accumulate(text_sizes.begin(), text_sizes.end(), number_of_texts);

if (number_of_texts == text_size)
throw std::invalid_argument("A text collection that only contains empty texts cannot be indexed.");
Expand Down

0 comments on commit 4604f1b

Please sign in to comment.