Skip to content

ete3: allow taxids in species tree input#163

Open
bernt-matthias wants to merge 5 commits intoEarlhamInst:mainfrom
bernt-matthias:topic/ete-species_tree_generator-taxids
Open

ete3: allow taxids in species tree input#163
bernt-matthias wants to merge 5 commits intoEarlhamInst:mainfrom
bernt-matthias:topic/ete-species_tree_generator-taxids

Conversation

@bernt-matthias
Copy link
Copy Markdown
Contributor

now one can submit a list of taxids as input (or even mixed)

@bernt-matthias bernt-matthias force-pushed the topic/ete-species_tree_generator-taxids branch from e3db6fd to 089b5dd Compare December 16, 2022 11:21
now one can submit a list of taxids as input
(or even mixed)
Gorilla gorilla gorilla Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Gorilla Gorilla gorilla
Pan troglodytes Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Pan Pan troglodytes
Homo sapiens Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Homo Homo sapiens
9606 Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Homo Homo sapiens
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure it's good that this and the following output files are changed, probably need to modify ete_lineage_generator.py to print the species name instead of the input taxid in the first column?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also noted this. The change is because the first column is just a copy of the input. I liked this because it gives a mapping between the original inputs and the ete outputs.

Should be easy to change it if you like. But we can also document it better.

In most use cases people will input either taxids of species names.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anilthanki Any opinion on this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @nsoranzo can we interpret no answer as positive feedback?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anilthanki is back from holidays, ping?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @anilthanki: any chance the get feedback? If not, can we just have a decision on how to continue here?

Copy link
Copy Markdown
Collaborator

@anilthanki anilthanki Jun 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bernt-matthias I did have a look before and tried to remember what It supposed to do but TBH its been long I worked on it so I couldnt really make a comment..

I will try to review it again this weekend and make a comment by monday

Apologies for this

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.

3 participants