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

Explain rooted tree vs. "biologically correct" root (curation tool) #524

Closed
jimallman opened this issue Jan 6, 2015 · 7 comments
Closed

Comments

@jimallman
Copy link
Member

Email from @rgazis:

Even though I rooted the tree (as in the publication) and selected the ingroup. Under trees it says "tree root: arbitrary (not biologically correct)". The same happens with the other studies.

image

@jar398 jar398 added this to the William S. Burroughs's birthday milestone Jan 22, 2015
jimallman added a commit that referenced this issue Jan 26, 2015
This is important for "arbitrary" rooting, since curators are
surprised to see this setting after choosing a tree's root.
Addresses #524.
@jimallman
Copy link
Member Author

To make this more clear, I've made the root-setting clickable in the tree list (this is subtle, since it's not really an error):
clickable-root-setting

Clicking this brings up the tree-popup to the Properties tab. If the user is just viewing the study, this shows an expanded explanation of arbitrary rooting:

screen shot 2015-01-26 at 3 05 02 pm

A curator who's editing the tree will see the root-setting widget highlighted as we've done for internal node labeling elsewhere:

screen shot 2015-01-26 at 3 08 05 pm

@jimallman
Copy link
Member Author

This is available for review on devtree. @rgazis, does this address your concerns?

@jar398
Copy link
Member

jar398 commented Jan 28, 2015

I'm not too keen on "arbitrary". In fact the root may not be arbitrary - it could actually be the correct one by coincidence, and it is at least likely to b one that is chosen methodically (if 'incorrectly'), not arbitrarily, by an algorithm. I would say in this situation that the correct (biological) root is unknown (to the app), not recorded (in phylesystem), not available (to us), or not curated (by any of our curators). This leads to something like

Biological root is
o confirmed by curator
o not confirmed (displayed root could be arbitrary)

@jimallman
Copy link
Member Author

not confirmed (displayed root could be arbitrary)

Thanks, will do. As a bonus, this more accurately reflects the logic of the app.

jimallman added a commit that referenced this issue Jan 28, 2015
@jimallman
Copy link
Member Author

These changes are live on devtree for review.

@jimallman
Copy link
Member Author

NOTE that this is causing a problem on devtree. Tree popup disappears immediately from this URL:
https://devtree.opentreeoflife.org/curator/study/edit/694?tab=trees

@jimallman
Copy link
Member Author

Fixed, merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants