Svenja Mehringer

Results 80 comments of Svenja Mehringer

please do not merge yet. I think a better deprecation warning for the record is needed.

@eseiler Can you please have a look at `sam_file/record.hpp` again

This is done with #3064

This issue will be automatically solved once we remove the `seqan3::field::alignment` from the sam_file. See related PRs: * #3057 (Adds a function `seqan3::alignment_from_cigar` that lets you create the alignment after...

Hi @Sawwave, Thanks for your detailed response which shows how to solve the problem (although not in a nice way because a nice way is not possible). Bad news, we...

We are also tracking MSA design decisions here: https://github.com/seqan/product_backlog/issues/104

Hm on my machine with our `format_fastq_benchmark` the changes don't prodocue much faster code. Could be a bad benchmark and according to hannes it also has a fluctuation depending on...

hm.. you think we should merge it? I have no strong feelings. I just had a 3% speedup but its more than nothing of course.

Hi @JanSiebert, this question seems resolved to me. Feel free to reopen the issue if something is missing for you.

Hi @cbielow, When talking to @eseiler, we concluded that this is not possible with the current design of the FM index in seqan3. The FM index can only perform exact...