Simon Gog
Hi Daniel, thanks for letting me know. Best, Simon
@tsnorri Thanks for the comment. Unftly, I was also not able to reproduce the issue on my machine.
csa_sada is (as far as I remember) the only class which is not thread-safe in the read-only scenario, since it stores some decoded integers in a cache. We will definitely...
For a quick fix, you would have to replace the buffer m_psi_buf by a local variable in the rank_bwt function: https://github.com/simongog/sdsl-lite/blob/master/include/sdsl/csa_sada.hpp#L330
Sorry, you are right; select on `wt_int` and `wm_int` are also not thread safe (a `grep mutable include/sdsl/*.hpp` ensured that). Here the two variables `m_path_off` and `m_path_rank_off` should be made...
No, the size of the vector is log(alphabet size). So it should be not a problem ;) A really efficient solution is to implement a version of select which gets...
great
Hi Erik, could you please open a pull request with your changes to wm_int and wt_int? Thanks! Simon ps: We have a thread-safe redesign of the other non-thread safe part...
@ekg, thanks for the pull request. The branch which makes csa_sada thread safe is not yet merged. We did a major redesign of `csa_sada` and will it merge in the...
Hi Diego, wow, thanks for reviewing the code so carefully. As @eloj already mentioned, the code should be fast and intelligible. In the past I discovered that many of these...