Opened 12 years ago

Closed 11 years ago

#10961 closed enhancement (fixed)

Add Lie bracket operation in Rings()

Reported by: Nicolas M. Thiéry Owned by: Alex Ghitza
Priority: major Milestone: sage-4.7.1
Component: algebra Keywords:
Cc: Merged in: sage-4.7.1.alpha0
Authors: Nicolas M. Thiéry Reviewers: Anne Schilling
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Anne Schilling)

From the documentation:

    sage: F = AlgebrasWithBasis(QQ).example(); F
    An example of an algebra with basis: the free algebra on the generators ('a', 'b', 'c') over Rational Field
    sage: a,b,c = F.algebra_generators()
    sage: F.bracket(a,b)
    B[word: ab] - B[word: ba]

Apply trac_10961-lie_bracket_in_rings-nt.patch

Attachments (2)

wifi (17 bytes) - added by Nicolas M. Thiéry 12 years ago.
Please delete this file!
trac_10961-lie_bracket_in_rings-nt.patch (1.5 KB) - added by Nicolas M. Thiéry 12 years ago.
Final version

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by Anne Schilling

Reviewers: Anne Schilling
Status: newneeds_work

The patch implements the Lie bracket and it would be good to merge it into sage soon. However, the tests in /categories/rings.py line 70 do not pass due to the misprint {{

sage: a,b,c = FF.algebra_generators()

}} should be {{

sage: a,b,c = F.algebra_generators()

}}

comment:2 in reply to:  1 Changed 12 years ago by Nicolas M. Thiéry

Status: needs_workneeds_review
Summary: Add Lie bracket operation in RingsAdd Lie bracket operation in Rings()

Replying to aschilling:

The patch implements the Lie bracket and it would be good to merge it into sage soon. However, the tests in /categories/rings.py line 70 do not pass due to the misprint {{

sage: a,b,c = FF.algebra_generators()

}} should be {{

sage: a,b,c = F.algebra_generators()

}}

Shoot; I had noticed this, and got distracted. Thanks for the reminder! Fixed!

Changed 12 years ago by Nicolas M. Thiéry

Attachment: wifi added

Please delete this file!

Changed 12 years ago by Nicolas M. Thiéry

Final version

comment:3 Changed 12 years ago by Nicolas M. Thiéry

Description: modified (diff)

comment:4 Changed 12 years ago by Anne Schilling

Status: needs_reviewpositive_review

Looks good!

comment:5 Changed 12 years ago by Nicolas M. Thiéry

Thanks for the review!

comment:6 Changed 12 years ago by Anne Schilling

Description: modified (diff)

comment:7 Changed 11 years ago by Jeroen Demeyer

Milestone: sage-4.7sage-4.7.1

comment:8 Changed 11 years ago by Jeroen Demeyer

Merged in: sage-4.7.1.alpha0
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.