Opened 11 years ago
Last modified 6 years ago
#451 new defect
'Move groups..' changes root
Reported by: | westram | Owned by: | devel |
---|---|---|---|
Priority: | major | Milestone: | wishlist |
Component: | ARB_NTREE | Version: | SVN |
Keywords: | Cc: |
Description (last modified by westram)
Status quo:
Method used to set root is very naive (see highlighted code):
- it is set to those 2 target-subtrees that match son-of-root-subtrees of source-. order is random. works ok if trees are similar. doesn't work at all if trees differ significantly.
- if group insertion could handle keeled groups (resp. keeled target subtrees), it would not be necessary to change root at all. Unwanted effects (by not doing so):
- if both sons of source-root are a group
⇒ results differ depending on target root position: sometimes one group is skipped, sometimes both get inserted - if one son of source-root is a group
⇒ group is (sometimes) placed at wrong son-of-root in destination
- if both sons of source-root are a group
Wanted fixes:
- root should be corrected using method implemented with #449 (only perform optionally)
- insert keeled group if it fits better (see arb documentation for meaning of keeled groups)
- if both sons of root are groups, they should match the correct subtrees (keeled or not) since implementation of #780. When function can handle keeled groups correctly
⇒ check problem again
e.g. happens with LTPs132_SSU.arb when root is changed:
Group 'Bacteria' skipped (got superseded by group 'Archaea'; same best target positions)
- if both sons of root are groups, they should match the correct subtrees (keeled or not) since implementation of #780. When function can handle keeled groups correctly
Change History (11)
comment:1 Changed 11 years ago by westram
- Description modified (diff)
comment:2 Changed 8 years ago by westram
- Description modified (diff)
comment:3 Changed 8 years ago by westram
- Description modified (diff)
comment:4 Changed 8 years ago by westram
- Description modified (diff)
comment:5 Changed 8 years ago by westram
- Description modified (diff)
- Milestone set to wishlist
comment:6 Changed 8 years ago by westram
- Description modified (diff)
comment:7 Changed 7 years ago by westram
- Priority changed from normal to major
comment:8 Changed 7 years ago by westram
- Description modified (diff)
comment:9 Changed 6 years ago by westram
- Description modified (diff)
comment:10 Changed 6 years ago by westram
- Description modified (diff)
comment:11 Changed 6 years ago by westram
- Description modified (diff)
- Summary changed from 'Move node info' (TreeAdmin) changes root to 'Move groups..' changes root
Note: See
TracTickets for help on using
tickets.
updated ticket while working on #780