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

Yara Index Memory alignment issue #2393

Open
svnbgnk opened this issue Sep 16, 2019 · 1 comment
Open

Yara Index Memory alignment issue #2393

svnbgnk opened this issue Sep 16, 2019 · 1 comment
Assignees
Labels

Comments

@svnbgnk
Copy link

svnbgnk commented Sep 16, 2019

Due to copying a dna5 alphabet into a dna4 index the resulting indexing data structures are no longer completely correct. This will affect the mapping process (backtracking) in extremly rare cases.
https://github.com/seqan/seqan/blob/f5f658343c366c9c3d44ba358ffc9317e78a09ed/apps/yara/indexer.cpp#L255-258.

I fixed this issue on a branch in Dream-Yara here:
svnbgnk/dream_yara@a02684a
I will submit a PR for Yara soon.

@svnbgnk svnbgnk self-assigned this Sep 16, 2019
@svnbgnk svnbgnk added the bug label Sep 16, 2019
@rrahn
Copy link
Contributor

rrahn commented Feb 9, 2020

So the silent conversion of 'N' to 'A' when converting dna5 to dna4 will cause this issue?
In general, would mapmap be a complete replacement of Yara? If so, then I would continue to apply this fix to SeqAn and focus on mapmap being ported to SeqAn3. Otherwise, if we are porting yara to SeqAn3 we can also make the fix there (if relevant).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants