mag_metromap_simple_light_v2-annotated.png
# graphics
j
mag_metromap_simple_light_v2-annotated.png
😍 3
m
Nice! Section text is not aligned with number. Don't like the dangling double node with centrifuge but I think we already discussed this I would flip the lower half, so that Multiqc is on the left.
j
1. Which section text? 2. Yeah, I'm sticking with this - unless you mean you prefer the line to go through the middle of station (the 'neck' so to say) 3. I've opted against this actually although I tried this, going from left to right is easier to read, and it helps with seaparation of the two major 'sections' of the pipeline (assembly vs binning), unless you have another particular reason other than you don't like the long line in the middle that tracks abck?
s
Wooooow......this looks so much better than v1 👏
❤️ 1
no.4 - how to interpret megahit/spades ? default or opt-in ?
j
Ah, the bold text indicates what executes by default
So in that case SPAdes hbyrid is not defualt
s
and ur turning off ancient dna by default now ? as I think its turned on by default currently right ?
j
No it's off by default?
s
ur right...its off by default
u wanna change
Assembled contigs
as contigs/scaffolds for future or we ll do that when we introduce that param ?
how about making the lines go through the default circles whereever applicable ?
j
1. When is a parameter. Or I could say assembled sequences? 2. This will make the thing too busy as there isn't enough space between the multi station circles. And it's not meant to be 100 accurate
Unless you think it's unclear
s
nah i think its fine....if any we ll propose when we PR our changes
👍 1
j
@mashehu you were right, a couple of them were indeed not aligned, fixed!