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.

Sign up for GitHub

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

Jump to bottom

Fix deprecated non-integer arguments to randrange #33770

Closed
mezzarobba opened this issue Apr 29, 2022 · 6 comments
Closed

Fix deprecated non-integer arguments to randrange #33770

mezzarobba opened this issue Apr 29, 2022 · 6 comments
Labels
r: duplicate t: bug

Comments

@mezzarobba
Copy link
Member

From src/sage/rings/polynomial/real_roots.pyx:

sage: from sage.rings.polynomial.real_roots import *
sage: bp = mk_ibpi([50, 20, -90, -70, 200], error=5)
sage: bp1, bp2, _ = bp.try_rand_split(mk_context(), None)
<ipython-input-1-a5287684725f>:3: DeprecationWarning: non-integer arguments to randrange() have been deprecated since Python 3.10 and will be removed in a subsequent version
  bp1, bp2, _ = bp.try_rand_split(mk_context(), None)
sage: bp1
<IBP: (50, 30, -21, -51, -20) + [0 .. 6) over [0 .. 325/512]>

CC: @orlitzky

Component: misc

Author: Marc Mezzarobba

Branch/Commit: u/mmezzarobba/33770 @ 9dd780d

Issue created by migration from https://trac.sagemath.org/ticket/33770
@mezzarobba mezzarobba added this to the sage-9.6 milestone Apr 29, 2022
@mezzarobba
Copy link
Member Author

Commit: 9dd780d

@mezzarobba
Copy link
Member Author

Branch: u/mmezzarobba/33770

@mezzarobba
Copy link
Member Author

New commits:

9dd780d#33770 fix deprecation warning

@mkoeppe
Copy link
Member

mkoeppe commented Apr 29, 2022

comment:2

see also #33234

@mezzarobba mezzarobba removed this from the sage-9.6 milestone Apr 30, 2022
@slel

This comment has been minimized.

Sign in to view
@slel slel changed the title deprecation warning with recent Python Fix deprecated non-integer arguments to randrange May 16, 2022
@fchapoton
Copy link
Contributor

comment:5

closing as duplicate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
r: duplicate t: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@slel @mezzarobba @fchapoton @mkoeppe