-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
olgabot/2022 jun comments #12
olgabot/2022 jun comments #12
Conversation
olgabot
commented
Jun 18, 2022
- Suggestion to add intro sentence to abstract
- Questions and comments on introduction
Got about 1.5 figures in so far, will continue soon! Questions/comments from inline HTML comments:
|
Hi! |
Hahah hello! Didn't mean to tag you there, sorry for the spam. Meant to be referencing a particular figure in the paper 😅 |
Made some more comments! Here are all the comments/questions/suggestions, listed out below:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for all of your comments Olga! I removed the inline comments and made issues/notes/or comments i'll add to the conversation. I also addressed a few typos or built on your suggestions as comments...hoping I can commit those into this PR after I publish this review.
Ok and some answers to you inline comments!
That's a really good question. I think it could mean the number of distinct sequences observed across all the genomes that were looked at, but typically it refers to the number of genes.
A pangenome can be considered all of the genomes in the world (although it not really possible to exhaustively sample all genomes in the world), but the openness or closed-ness of a pangenome is a property of the eco-evo strategy of that group of organisms. Closed pangenomes are usually associated with organisms that have a very small niche breadth.
Eh...it's sort of unclear. Genome is a bit more exact, as strain can have many definitions.
Good catch -- it's other strains from different species that get scooped in because assembly graph queries retrieve things down to about ~93% ANI. I updated the language to make this more clear.
This is probably true...I'm going to leave it for now, and if necessary I'll go back and benchmark.
Love this idea! Will update. Also note I made issues #13, #14, and #15 to account for the (potentially) missing figures. There's a chance I may preprint without these figures, and then will think about adding them to the supplement before submitting for publication. Lastly, I will either merge this branch and then update the figures (add lines of fit, t test results, and contingency table for legend) and add more details to some sections of the methods, or will push more changes to this branch. Thank you again so much olga! your comments were 💯 |
[ci skip] This build is based on be300d6. This commit was created by the following CI build and job: https://github.com/dib-lab/2021-paper-metapangenomes/commit/be300d6f0a8d50479352e52c9fc6a5d1c99f16c6/checks https://github.com/dib-lab/2021-paper-metapangenomes/actions/runs/2569364395
[ci skip] This build is based on be300d6. This commit was created by the following CI build and job: https://github.com/dib-lab/2021-paper-metapangenomes/commit/be300d6f0a8d50479352e52c9fc6a5d1c99f16c6/checks https://github.com/dib-lab/2021-paper-metapangenomes/actions/runs/2569364395
Yes, this is something that I think Taylor and Tessa came up with and just started using, and it's SO MUCH CLEARER than using "strain"! 🎉 Like "is it a different strain or not?" Well who knows, but it's definitely a different genome sequence, so 🤷 |