more scalar conversions

[videlec]

Following #10147, we extend conversion to scalars for polynomials, multi-polynomials and fraction fields.

The attached branch allows the following construction

sage: x = polygen(QQ)
sage: A.<u> = NumberField(x^3 - 2)
sage: A((x + 3) / (2*x - 1))
14/15*u^2 + 7/15*u + 11/15

See the original report from asksage.

Component: basic arithmetic

Author: Vincent Delecroix

Branch: e40555a

Reviewer: Travis Scrimshaw

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

[videlec]

Branch: u/vdelecroix/31716

[videlec]

Author: Vincent Delecroix

[videlec]

New commits:

76fdce3

31716: more scalar conversions

[videlec]

Commit: 76fdce3

[sagetrac-git]

Changed commit from 76fdce3 to 9dbc024

[sagetrac-git]

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

e2a22cb	31716: more scalar conversions
9dbc024	31716: fix doctests

[sagetrac-git]

Changed commit from 9dbc024 to 4e245da

[sagetrac-git]

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

4e245da

31716: more doctests

[tscrim]

comment:7

LGTM except you lost the doctest for MPolynomial._integer_. Once you add one, you can set a positive review.

[tscrim]

Reviewer: Travis Scrimshaw

[sagetrac-git]

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

779b1ad

more tests

[sagetrac-git]

Changed commit from 4e245da to 779b1ad

[sagetrac-git]

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

e40555a

31716: remove MPolynomial._integer_

[sagetrac-git]

Changed commit from 779b1ad to e40555a

[videlec]

comment:10

Thanks for spotting that. My goal was actually to remove the _integer_ method (done in e40555a). I copied over the tests in _scalar_conversion (779b1ad).

[tscrim]

comment:11

Once the patchbot comes back green, positive review.

[videlec]

comment:12

green. Thanks.

[vbraun]

Changed branch from u/vdelecroix/31716 to e40555a

[mezzarobba]

Changed commit from e40555a to none

[mezzarobba]

comment:14

I suspect (but have not checked) that #32078 was caused by the changes here.