Skip to content

Commit e79f6ff

Browse files
author
Release Manager
committed
Trac #33966: KnotInfo interface returns wrong braid_notation for the unknot
The original database does not provide a value for the `braid_notation` of the unknot. Thus this value has been set individually in the interface, but unfortunately this value corresponds to a disjoint union of two unknots. This bug remained unnoticed because the construction of a link from a braid in Sage erases all additional unknots. URL: https://trac.sagemath.org/33966 Reported by: soehms Ticket author(s): Sebastian Oehms Reviewer(s): Travis Scrimshaw
2 parents dd0ed53 + 99345e1 commit e79f6ff

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/sage/knots/knotinfo.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -611,14 +611,21 @@ def braid_notation(self, original=False):
611611
'{3, {-2, -2, -1, 2, -1}}'
612612
sage: L[L.items.braid_notation_old]
613613
'{4, {1, -2, 3, -2, -1, -2, -3, -2}}'
614+
615+
TESTS:
616+
617+
Check that :trac:`33966` is fixed::
618+
619+
sage: KnotInfo.K0_1.braid_notation()
620+
(1,)
614621
"""
615622
braid_notation = self[self.items.braid_notation]
616623
if original:
617624
return braid_notation
618625

619626
if not braid_notation:
620627
# don't forget the unknot
621-
return (1, -1)
628+
return (1, )
622629

623630
braid_notation = eval_knotinfo(braid_notation)
624631
if type(braid_notation) is list:

0 commit comments

Comments
 (0)