Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
shorter error message per reviewer's request
Browse files Browse the repository at this point in the history
  • Loading branch information
mantepse committed Aug 13, 2018
1 parent 9835998 commit f54b339
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/sage/combinat/diagram_algebras.py
Expand Up @@ -1454,12 +1454,13 @@ def symmetric_diagrams(self, l=None, perm=None):
sage: bd.symmetric_diagrams(l=1, perm=[2,1])
Traceback (most recent call last):
...
NotImplementedError: symmetric_diagrams is only implemented for Brauer diagrams of integer order, not for order 3/2
NotImplementedError: only implemented for integer order, not for order 3/2
"""
# perm = permutation on free nodes
# l = number of arcs
if self.order not in ZZ:
raise NotImplementedError("symmetric_diagrams is only implemented for Brauer diagrams of integer order, not for order %s" %(self.order))
raise NotImplementedError("only implemented for integer order,"
" not for order %s" % (self.order))
n = ZZ(self.order)
if l is None:
l = 0
Expand Down Expand Up @@ -1511,10 +1512,11 @@ def from_involution_permutation_triple(self, D1_D2_pi):
sage: bd.from_involution_permutation_triple([[[1,2]],[[3,4]],[2,1]])
Traceback (most recent call last):
...
NotImplementedError: from_involution_permutation_triple is only implemented for Brauer diagrams of integer order, not for order 5/2
NotImplementedError: only implemented for integer order, not for order 5/2
"""
if self.order not in ZZ:
raise NotImplementedError("from_involution_permutation_triple is only implemented for Brauer diagrams of integer order, not for order %s" %(self.order))
raise NotImplementedError("only implemented for integer order,"
" not for order %s" % (self.order))
try:
(D1,D2,pi) = tuple(D1_D2_pi)
except ValueError:
Expand Down

0 comments on commit f54b339

Please sign in to comment.