#15505: Braid Group construction misses some relations
--------------------------------+----------------------------
       Reporter:  mmarco        |        Owner:
           Type:  defect        |       Status:  needs_review
       Priority:  major         |    Milestone:  sage-5.13
      Component:  group theory  |   Resolution:
       Keywords:  braid groups  |    Merged in:
        Authors:  mmarco        |    Reviewers:
Report Upstream:  N/A           |  Work issues:
         Branch:                |       Commit:
   Dependencies:                |     Stopgaps:
--------------------------------+----------------------------

Comment (by tscrim):

 The if statement is always true since `range(a, b)` is `[a, a+1, ..., b-2,
 b-1]` (or you can think of it as the integer points in th internal `[a,
 b)`). So I think this should work:
 {{{#!python
 for i in range(1, n):
      rels.append(free_group([i, i+1, i, -i-1, -i, -i-1]))
      for j in range(i+2, n+1):
          rels.append(free_group([i, j, -i, -j]))
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/15505#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to