Taxa substitution failed due to unknown error

Bug #1703354 reported by Fabian Kellner on 2017-07-10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Supertree Toolkit
Jon Hill

Bug Description

When trying to substitute taxa this error is displayed:

Failed to substitute taxa due to an unknown error. Possibly a bug. Please report on Launchpad.
Traceback (most recent call last):
  File "C:\Python27\lib\site-packages\supertree_toolkit-2.0-py2.7.egg\stk_gui\", line 2297, in on_sub_taxa_sub_taxa_button
  File "C:\Python27\lib\site-packages\supertree_toolkit-2.0-py2.7.egg\stk\", line 1678, in substitute_taxa
  File "C:\Python27\lib\site-packages\supertree_toolkit-2.0-py2.7.egg\stk\", line 3265, in _swap_tree_in_XML
ValueError: invalid literal for int() with base 10: 'NO'

Jon Hill (jon-hill) wrote :

Thanks for the report. I'll take a look. Can you attach the Phyml and the substitutions file (or the substitution you were trying to do) here so I can try and replicate this?



Fabian Kellner (fabianlk) wrote :

Thanks for the quick reply! Files are attached.
- Fabian

Fabian Kellner (fabianlk) wrote :
Jon Hill (jon-hill) wrote :

Super. Thanks. I'll try and get this looked at this week.

Jon Hill (jon-hill) on 2017-07-12
Changed in supertree-toolkit:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Jon Hill (jon-hill)
Jon Hill (jon-hill) wrote :

Bug is caused by assuming tree names are in the form sourceName_n where n is an integer. This is a long-standing bug. Fix is to replace this code with more generic searches in _swap_tree_in_xml

Jon Hill (jon-hill) wrote :

Fix committed in:

Tried the substitution on the submitter's data and it worked ok producing fewer taxa, but some are now permutable trees (contain %1, etc).

Jon Hill (jon-hill) on 2017-07-12
Changed in supertree-toolkit:
status: Confirmed → Fix Committed
information type: Public → Private
information type: Private → Public
Fabian Kellner (fabianlk) wrote :

Thanks for the quick help John! The github link is not working (for me) though...

- Fabian

Jon Hill (jon-hill) wrote :

Sorry - I merged it into master and deleted the branch:

I'll try and create a new .deb file (and Windows/Mac releases) soon.

Fabian Kellner (fabianlk) wrote :

Thank you very much :)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers