Add method to trim trailing zeros from IntegerVector

[trevorkarn]

Remove trailing zeroes from integer vectors using .trim

CC: @tscrim @trevorkarn @darijgr

Component: combinatorics

Keywords: gsoc2022 compositions integer-vector

Author: Trevor K. Karn

Branch/Commit: b537651

Reviewer: Travis Scrimshaw

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

[trevorkarn]

Commit: 12f7909

[trevorkarn]

Last 10 new commits:

e973f7d	Assertion -> ValueError
a57b2d0	Add tests and remove Stanley s.f.
28286da	Add methods to interact with polyominos/compositions
75ce24e	Change to list
8c3cb4e	Add type-A algorithm
4a6b2f9	Rewrite to pass to Permutations
9c2a107	Initial commit of KeyPolynomial
5899abf	Fix divided difference
24098b5	Fix expand
12f7909	Add .trim method to compositions

[trevorkarn]

Branch: u/tkarn/trim-34435

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

6c6300b

Add .trim method to compositions

[sagetrac-git]

Changed commit from 12f7909 to 6c6300b

[trevorkarn]

Changed keywords from gsoc2022 compositions to gsoc2022 compositions integer-vector

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

ab673bd

Add trim to integer vector

[sagetrac-git]

Changed commit from 6c6300b to ab673bd

[trevorkarn]

comment:7

Moved from compositions to integer vectors because Sage Compositions should not have zero entries.

[tscrim]

comment:8

This will not quite work as the parent might require the integer vector to have a fixed length. You should simply return an element in all integer vectors. You can also more directly construct the elements by P.element_class(P, foo, check=False) (which will also make this faster) since you can assume you started with an integer vector.

[sagetrac-git]

Changed commit from ab673bd to b537651

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

b537651

Fix parent

[trevorkarn]

comment:10

Replying to @tscrim:

This will not quite work as the parent might require the integer vector to have a fixed length. You should simply return an element in all integer vectors. You can also more directly construct the elements by P.element_class(P, foo, check=False) (which will also make this faster) since you can assume you started with an integer vector.

Makes senses, thanks!

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:11

Thanks. LGTM.

[vbraun]

Changed branch from u/tkarn/trim-34435 to b537651