Segfault when floor dividing in infinite polynomial ring

[trevorkarn]

sage: R.<z> = InfinitePolynomialRing(QQ)
sage: (z[4] + z[5])//(z[4] + z[5])
------------------------------------------------------------------------
(no backtrace available)
------------------------------------------------------------------------
Unhandled SIGSEGV: A segmentation fault occurred.
This probably occurred because a *compiled* module has a bug
in it and is not properly wrapped with sig_on(), sig_off().
Python will now terminate.
------------------------------------------------------------------------
/Users/trevorkarn/Applications/sage/src/bin/sage-python: line 2: 49123 Segmentation fault: 11  sage -python "$@"

CC: @tscrim @trevorkarn

Component: algebra

Keywords: gsoc2022 infinite-polynomial divide

Author: Trevor K. Karn

Branch/Commit: 77cbf42

Reviewer: Travis Scrimshaw

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

[trevorkarn]

New commits:

1f3e285

Add _floordiv_

[trevorkarn]

Branch: u/tkarn/divide-34535

[trevorkarn]

Commit: 1f3e285

[tscrim]

comment:2

Do we need to implement checks for when the polynomials might not be in the same ring? Cf. _add_.

[sagetrac-git]

Changed commit from 1f3e285 to 1f0a3d0

[sagetrac-git]

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

1f0a3d0

Update floordiv

[trevorkarn]

comment:4

Updated to be modeled after _add_.

[sagetrac-git]

Changed commit from 1f0a3d0 to 77cbf42

[sagetrac-git]

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

77cbf42

PEP8 compliance

[trevorkarn]

comment:6

Ok things should now be good.

[tscrim]

comment:7

Green bot => positive review.

[trevorkarn]

comment:8

There is one failure on the bot

**********************************************************************
File "src/sage/schemes/toric/sheaf/klyachko.py", line 951, in sage.schemes.toric.sheaf.klyachko.KlyachkoBundle_class.random_deformation
Failed example:
    Vtilde.cohomology(dim=True, weight=(0,))
Expected:
    (1, 0)
Got:
    (0, 0)
**********************************************************************

Which seems unrelated to this ticket.

[tscrim]

comment:9

Indeed.

[tscrim]

Reviewer: Travis Scrimshaw

[vbraun]

Changed branch from u/tkarn/divide-34535 to 77cbf42