Ticket #13796 (closed defect: fixed)

Opened 7 months ago

Last modified 6 months ago

fix small bug in Steenrod algebra multiplication

Reported by: jhpalmieri Owned by: AlexGhitza
Priority: major Milestone: sage-5.6
Component: algebra Keywords: steenrod algebra
Cc: Work issues:
Report Upstream: N/A Reviewers: André Apitzsch
Authors: John Palmieri Merged in: sage-5.6.beta1
Dependencies: Stopgaps:

Description

The attached one-line patch (plus doctests) fixes a small bug in multiplication in the Steenrod algebra. It should be easy to review even if you don't know anything about the Steenrod algebra; the code should have defined "F" to be "GF(p)", but it didn't, so "F" is not defined, so it causes an error.

Attachments

trac_13796-steenrod.patch Download (965 bytes) - added by jhpalmieri 7 months ago.

Change History

Changed 7 months ago by jhpalmieri

comment:1 Changed 7 months ago by jhpalmieri

  • Status changed from new to needs_review

comment:2 Changed 7 months ago by aapitzsch

  • Status changed from needs_review to positive_review
  • Reviewers set to André Apitzsch

I haven't check whether your doc test is correct. Anyway all tests passed. So I give this a positive review.

comment:3 Changed 6 months ago by jdemeyer

  • Status changed from positive_review to closed
  • Resolution set to fixed
  • Merged in set to sage-5.6.beta1
Note: See TracTickets for help on using tickets.