empty partition causes error for up_list

[saliola]

sage: p = Partition([])
sage: p.up_list()
Traceback ...
...
IndexError: list index out of range

Depends on #11300

CC: @sagetrac-sage-combinat

Component: combinatorics

Keywords: combinat, partitions, days30

Author: Jason Bandlow

Reviewer: Franco Saliola

Merged: sage-4.7.1.alpha1

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

[jbandlow]

Attachment: trac_11301-uplist_fails_on_empty_partition-jb.patch.gz

[jbandlow]

Author: Jason Bandlow

[jbandlow]

Dependencies: #11300

[jbandlow]

comment:2

Notice the dependency on 11300.

[saliola]

Reviewer: Franco Saliola

[hivert]

comment:4

Replying to @saliola:

Shouldn't we rename up_list to something more meaningful ? What about young_lattice_successors ?

Florent

[nthiery]

comment:5

Replying to @hivert:

Replying to @saliola:

Shouldn't we rename up_list to something more meaningful ? What about young_lattice_successors ?

Ah, this is what up_list is about! I had no clue before that
(from the name).

+1 for renaming (with deprecation of up_list), either in this
patch or a followup.

For the name, for consistency with other spots (like
bruhat_upper_covers), we should use a name like
young_upper_covers or young_lattice_upper_covers.

You may want to raise a discussion on sage-combinat-devel to choose
between the two.

Cheers,
Nicolas

[jdemeyer]

Merged: sage-4.7.1.alpha1