Skip to content

Commit 4604f1b

Browse files
authored
Merge pull request #2756 from SGSSGene/fix/large_bi_fm_index
[FIX] Constructing large fm_indices
2 parents 0e28954 + c9a515f commit 4604f1b

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

CHANGELOG.md

+9
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,15 @@ The following API changes should be documented as such:
2424
If possible, provide tooling that performs the changes, e.g. a shell-script.
2525
-->
2626

27+
# 3.1.0
28+
29+
## Notable Bug-fixes
30+
31+
#### Search
32+
33+
* Resolved an issue that prevented the FM-Index from being constructed correctly for inputs larger than 4 GiB
34+
([\#2756](https://github.com/seqan/seqan3/pull/2756)).
35+
2736
# 3.0.3
2837

2938
Note that 3.1.0 will be the first API stable release and interfaces in this release might still change.

include/seqan3/search/fm_index/fm_index.hpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ class fm_index
293293
size_t const number_of_texts{text_sizes.size()};
294294

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

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

0 commit comments

Comments
 (0)