Closed jeffkaufman closed 7 months ago
I don't think I have enough context here to do a meaningful review. I haven't worked with refseq or the NCBI API. General note is that I would find these easier to review with even a small number of comments describing what a function or call does :P
I can look at it today or tomorrow
On Fri, 17 Nov 2023 at 1:51 PM Lenni Justen @.***> wrote:
I don't think I have enough context here to do a meaningful review. I haven't worked with refseq or the NCBI API. General note is that I would find these easier to review with even a small number of comments describing what a function or call does :P
— Reply to this email directly, view it on GitHub https://github.com/naobservatory/mgs-pipeline/pull/37#issuecomment-1816927256, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN7ASMRZUIIA2RA2VJ3KFPDYE6W4ZAVCNFSM6AAAAAA7P7AGV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJWHEZDOMRVGY . You are receiving this because your review was requested.Message ID: @.***>
When we run bowtie we get genome ids, but usually we actually want taxids. Make a map so it's easy to get back to taxids.