Opened 7 years ago

Closed 7 years ago

#13893 closed defect (fixed)

stabilizer doesn't work with custom domain

Reported by: jason Owned by: AlexGhitza
Priority: major Milestone: sage-5.6
Component: algebra Keywords:
Cc: Merged in: sage-5.6.beta3
Authors: Jason Grout Reviewers: Mike Hansen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Right now, there is a typo in the stabilizer method that means it won't work with the permutation group has a custom domain.

Attachments (1)

13893-stabilizer-domain.patch (1.6 KB) - added by jason 7 years ago.

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by jason

comment:1 Changed 7 years ago by jason

  • Status changed from new to needs_review

comment:2 Changed 7 years ago by mhansen

  • Reviewers set to Mike Hansen
  • Status changed from needs_review to positive_review

Looks good to me. Nice catch.

comment:3 Changed 7 years ago by jason

Good to hear from you, Mike! Thanks for the review.

comment:4 Changed 7 years ago by jdemeyer

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