pmelsted / bifrost

Bifrost: Highly parallel construction and indexing of colored and compacted de Bruijn graphs
BSD 2-Clause "Simplified" License
204 stars 25 forks source link

Fix memory error in multithreaded query #46

Closed sebschmi closed 3 years ago

sebschmi commented 3 years ago

Apparently, the buffer_res was created with the number of threads in the multithreaded query. In the single-threaded query it is created with thread_seq_buf_sz, so I used that same variable here and it seems to work.

Fixes #45

GuillaumeHolley commented 3 years ago

Hi @sebschmi,

I apologize for the long delay in accepting the PR, I really appreciate that you not only identified the issue but also found a fix for it. Thank you for using Bifrost and for your time spent on it.

Guillaume