Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a multi-partition implementation to sage #10630

Closed
pabryan mannequin opened this issue Jan 14, 2011 · 9 comments
Closed

Add a multi-partition implementation to sage #10630

pabryan mannequin opened this issue Jan 14, 2011 · 9 comments

Comments

@pabryan
Copy link
Mannequin

pabryan mannequin commented Jan 14, 2011

As discussed here[1], multi-partitions would be a useful addition to sage. I have an initial implementation soon to be uploaded.

[1] http://groups.google.com/group/sage-combinat-devel/browse_thread/thread/aefcc71ad217d023/439360dc6b2641c5

Apply: attachment: trac_10630-vector_partition-ap.patch

Component: combinatorics

Keywords: multipartition multi-partition multi-partite multipartite partition

Author: Amritanshu Prasad

Reviewer: Travis Scrimshaw

Merged: sage-5.12.beta0

Issue created by migration from https://trac.sagemath.org/ticket/10630

@pabryan pabryan mannequin added this to the sage-5.11 milestone Jan 14, 2011
@pabryan pabryan mannequin added c: combinatorics labels Jan 14, 2011
@pabryan pabryan mannequin self-assigned this Jan 14, 2011
@pabryan
Copy link
Mannequin Author

pabryan mannequin commented Jan 21, 2011

Changed author from pabryan to Paul Bryan

@pabryan
Copy link
Mannequin Author

pabryan mannequin commented Feb 2, 2011

Implementation of multi-partitions

@EmmanuelJeanBriand
Copy link

comment:2

Attachment: trac_10630_multi_partitions.patch.gz

Some comments:

  • The functions for multipartitions do not seem to work correctly. Here is an incorrect (incomplete) result:
 
M=MultiPartitions([2,2])
for x in M.list(): print x
[[2, 2]]
[[1, 2], [1, 0]]
[[2, 1], [0, 1]]
[[1, 0], [1, 0], [0, 2]]
[[1, 1], [1, 0], [0, 1]]
[[2, 0], [0, 1], [0, 1]]
[[1, 0], [1, 0], [0, 1], [0, 1]]

Two multipartitions are missed: [[1,1],[1,1]] and [[2,0],[0,2]]

  • I wonder whether MultiPartitions is a good name, because it seems it has a different meaning in a near context. See for instance this paper by G. Andrews: http://es.scribd.com/doc/6625886/Multi-Partitions, or this other paper; http://www.maths.qmul.ac.uk/~mf/papers/weight2.pdf, or these slides: http://www.cs.clemson.edu/~goddard/MINI/2009/Swisher.pdf ."Vector partitions" may be safer (it is used for instance by George Andrews in his book on partitions).

  • Should one represent the Multipartitions as lists of vectors with integer coordinates rather than lists of lists ? (because sometimes one has to do the sum of the parts).

  • A few typos:

    • Line 6 of the file: "the" should not be repeated.
    • Line 9 of the file: replace "it's" with "its".
    • Line 63 of the file: replace "lexicographically order integer lists" with "lexicographically weakly decreasing lists of integers" or something better.
    • Also, one might replace the definition of multipartition as "equivalence class of tuples" (lines 4 to 10) with "multiset of tuples".

@amritanshu-prasad
Copy link
Mannequin

amritanshu-prasad mannequin commented Jul 10, 2013

Changed author from Paul Bryan to Amritanshu Prasad

@amritanshu-prasad

This comment has been minimized.

@amritanshu-prasad
Copy link
Mannequin

amritanshu-prasad mannequin commented Jul 10, 2013

comment:3

Vector partitions is now implemented. I am calling it VectorPartitions instead of MutliPartitions, to remove the conflict with the standard usage of the latter.

Apply: trac_10630-vector_partition-ap.patch

@amritanshu-prasad
Copy link
Mannequin

amritanshu-prasad mannequin commented Jul 10, 2013

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jul 11, 2013

comment:4

Attachment: trac_10630-vector_partition-ap.patch.gz

Looks good to me.

Apply: trac_10630-vector_partition-ap.patch

@jdemeyer
Copy link

jdemeyer commented Aug 2, 2013

Merged: sage-5.12.beta0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants