Skip to content
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

ete3: allow taxids in species tree input #163

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

bernt-matthias
Copy link
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)
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.py Outdated Show resolved Hide resolved
tools/ete/ete_species_tree_generator.xml Outdated Show resolved Hide resolved
@@ -29,7 +29,7 @@ Nomascus leucogenys Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Noma
Pongo abelii Eukaryota Chordata Mammalia Euarchontoglires Hominoidea Pongo Pongo abelii
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
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
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
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
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
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
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
Member

@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