Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#572 closed defect (fixed)

parsimony "add partial" produces unwanted structure

Reported by: westram Owned by: westram
Priority: major Milestone:
Component: ARB_PARSIMONY Version: SVN
Keywords: arb6.0-hotfix Cc:

Description

When adding partial sequences using ARB_PARSIMONY each is placed next to one selected full sequence. When multiple partials are added to the same full sequence, they are grouped together in one subtree and that subtree is placed as "brother" of the full sequence.

All inner branch lengths of that subtree should be zero and only leaf branches should have a length (=the "partial distance" between full and partial sequence).

This works for 2 or fewer partial species and fails for 3 or more (see attached screenshot).

Attachments (2)

added_partials.png (8.7 KB) - added by westram 10 years ago.
Shows wrong placed partials (next to RhoPy162 and RhoOp626)
fixed.png (8.3 KB) - added by westram 10 years ago.
result after fix

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by westram

Shows wrong placed partials (next to RhoPy162 and RhoOp626)

comment:1 Changed 10 years ago by westram

  • Owner changed from devel to westram
  • Status changed from new to _started

comment:2 Changed 10 years ago by westram

  • Resolution set to fixed
  • Status changed from _started to closed

by [12489]

Branchlength was not restored at insertion point, but at brother of full sequence. These two were the same for first and second partial added.

Changed 10 years ago by westram

result after fix

comment:3 Changed 9 years ago by westram

  • Keywords arb6.0-hotfix added

mark arb6.0.x hotfixes

Note: See TracTickets for help on using tickets.