Skip to content

Commit

Permalink
Small bug fix for avgSMS comparison
Browse files Browse the repository at this point in the history
  * when combining SRs (datasets), if each dataset theory prediction
    contains a single SMS, an averageSMS is assign to the dataset prediction.
  * when checking if all the averageSMS are the same (so we can assign a single
    SMS to the combination ) the comparison breaks
    if the avgSMS have distinct shapes (canonical names)
  * the fix includes a canonical name comparison to the avgSMS comparison,
    so it works even if the avgSMS have distinct shapes.
  • Loading branch information
andlessa committed Oct 9, 2024
1 parent f491597 commit b6b51bc
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions smodels/matching/clusterTools.py
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,9 @@ def __cmp__(self, other):

if not isinstance(other, TheorySMS):
return -1

if self.canonName != other.canonName:
return -1

for nodeIndex in self.nodeIndices:
nodeA = self.indexToNode(nodeIndex)
Expand Down

0 comments on commit b6b51bc

Please sign in to comment.