Is there a reason that `SAMTOOLS_MERGE` supports c...
# modules
a
Is there a reason that
SAMTOOLS_MERGE
supports capturing an outputted
csi
or
crai
index but not a
bai
?
f
Probably because bai is not the default option with
--write-index
. Started adding an input
val index_format
for lack of a better option (in e.g. https://github.com/nf-core/modules/blob/master/modules/nf-core/samtools/view/main.nf). But I think @Maxime Garcia was hoping for a better solution.
👀 1
m
you mean for the
unselected_index
?
I think mainly because it was somehow deleted when the index was fixed from
crsi
to
crai
wait
I was looking at samtools/view
so I would say simply forgotten, looking at git blame I can say that csi support and crai support was added independently
👍 1
a
Personally I feel all the
samtools
modules (and similarly
bcftools
modules) should have the same behaviour for handling indexes where possible, because the usability of
nf-core
modules, especially for newcomers, decreases considerably when such similar modules they have differing interfaces
this 2
m
TOTALLY AGREEING