You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to DendroPy Users
Line 5314 of treemodel.py
if not is_bipartitions_updated or not self.bipartitions_encoding:
should be
if not is_bipartitions_updated or not self.bipartition_encoding:
Paul Lewis
unread,
Aug 4, 2015, 10:55:14 AM8/4/15
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to DendroPy Users
Sorry, I accidentally posted this before I was finished typing. The misspelling of "self.bipartition_encoding" as "self.bipartitions_encoding" at line 5341 in treemodel.py leads to an error if the Tree function is_compatible_with_bipartition is used with is_bipartitions_updated=True.
Jeet Sukumaran
unread,
Aug 4, 2015, 10:58:41 AM8/4/15
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to dendrop...@googlegroups.com
Hi Paul,
Thanks for pointing this out. The second clause never gets executed in
the tests because the first always is true (in the tests). Testing is
tricky ...