Skip to content

Add ampliconstats to the main samtools man page.#1351

Merged
jkbonfield merged 1 commit intosamtools:developfrom
whitwham:minor_man_page_update
Dec 15, 2020
Merged

Add ampliconstats to the main samtools man page.#1351
jkbonfield merged 1 commit intosamtools:developfrom
whitwham:minor_man_page_update

Conversation

@whitwham
Copy link
Copy Markdown
Member

@whitwham whitwham commented Dec 9, 2020

Added the missing command.

@whitwham
Copy link
Copy Markdown
Member Author

whitwham commented Dec 9, 2020

Note, the man page on htslib.org needs updating as it is still on 1.10. After this is merged though.

@jkbonfield
Copy link
Copy Markdown
Contributor

jkbonfield commented Dec 9, 2020

Thanks for this.

I'm in two minds as to where this belongs in the ordering.

It could either be adjacent to ampliconclip, as the two form their own group of amplicon-based analysis, or immediately after stats as part of the overall statistics group (flagstats, idxstats, stats). I guess bedcov, depth, mpileup, coverage all form genome coordinate depth style plotting, which ampliconstats may also be said to share a common grouping. I guess this is why you put it there.

Between stats and bedcov feels like a natural transition between the two groups? (Which rather fortuitously just happen to be adjacent.)

No strong feelings though so feel free to argue your case. :-)

@whitwham
Copy link
Copy Markdown
Member Author

whitwham commented Dec 9, 2020

Yes, I was in two minds about where it should go. It seemed more stat like so I put it there. It could well have just been put at the end near ampliconclip. I have no really strong feelings about where it should go either.

@jkbonfield jkbonfield merged commit b246153 into samtools:develop Dec 15, 2020
@whitwham whitwham deleted the minor_man_page_update branch February 14, 2022 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants