Opened 9 years ago

Closed 9 years ago

#15151 closed enhancement (fixed)

Implement residue functions for partitions

Reported by: tscrim Owned by: sage-combinat
Priority: major Milestone: sage-5.13
Component: combinatorics Keywords:
Cc: sage-combinat Merged in: sage-5.13.beta0
Authors: Travis Scrimshaw Reviewers: Darij Grinberg
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by darij)

Implements methods which directly returns a given residue and iterators for addable/removable cells with a fixed residue.

Apply: trac_15151-partition_resides-ts.patch

Attachments (2)

trac_15151-rev-dg.patch (4.8 KB) - added by darij 9 years ago.
review patch (final, now with correct latex)
trac_15151-partition_resides-ts.patch (6.0 KB) - added by tscrim 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by tscrim

  • Authors set to Travis Scrimshaw
  • Status changed from new to needs_review

comment:2 Changed 9 years ago by darij

Nice little patch. As usual I took the liberty to improve the doc around it. If you are OK with my changes, mark this positive_review.

Changed 9 years ago by darij

review patch (final, now with correct latex)

Changed 9 years ago by tscrim

comment:3 Changed 9 years ago by tscrim

  • Reviewers set to Darij Grinberg
  • Status changed from needs_review to positive_review

I folded in your review patch and made one small change: \mathbb Z^2 -> \ZZ^2 to be consistent with the Sage latex macro. Thanks for doing the review Darij.

comment:4 Changed 9 years ago by tscrim

  • Description modified (diff)

For patchbot:

Apply: trac_15151-partition_residues-ts.patch

comment:5 Changed 9 years ago by darij

  • Description modified (diff)

For patchbot:

Apply: trac_15151-partition_resides-ts.patch

Last edited 9 years ago by darij (previous) (diff)

comment:6 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-5.12 to sage-5.13

comment:7 Changed 9 years ago by jdemeyer

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