Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#28002 closed defect (fixed)

py3: last test in tests/cmdline.py

Reported by: John Palmieri Owned by:
Priority: major Milestone: sage-8.9
Component: python3 Keywords:
Cc: Merged in:
Authors: John Palmieri Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 8712a2f (Commits, GitHub, GitLab) Commit: 8712a2fab3fbab9944a4a08fc670515bd0b39ff2
Dependencies: Stopgaps:

Status badges

Description

The fix is to ignore deprecation warnings about collections.abc. This already done in src/sage/all.py. At some point, this should be fixed throughout the Sage library, but that's for another ticket.

Change History (10)

comment:1 Changed 3 years ago by John Palmieri

Branch: u/jhpalmieri/cmdline-py3

comment:2 Changed 3 years ago by John Palmieri

Commit: 8712a2fab3fbab9944a4a08fc670515bd0b39ff2
Status: newneeds_review

New commits:

8712a2ftrac 28002: in sage-startuptime.py, ignore collections.abc deprecation

comment:3 Changed 3 years ago by John Palmieri

Regarding imports of collections.abc and the deprecation warning: it seems to come as much from places like matplotlib than from the actual Sage library. So maybe the problem will get fixed as we update the various Sage packages.

comment:4 Changed 3 years ago by Frédéric Chapoton

Reviewers: Frédéric Chapoton
Status: needs_reviewpositive_review

ok, thx. Is there a way to make sure that sage itself is not responsible for any of these warnings ?

comment:5 Changed 3 years ago by John Palmieri

Both

grep -R "import collections.*abc" src/sage src/sage_setup src/doc

and

grep -R "from collections.*import.*abc" src/sage src/sage_setup src/doc

come up empty. I tried turning off the warnings in src/sage/all.py, and when I ran a few doctests, I got a warning from matplotlib. I don't know what else to try to do to check.

comment:6 Changed 3 years ago by Frédéric Chapoton

Ok, then we will se later when we upgrade matplotlib.

By the way, do you know "git grep" ? Useful tool, in my opinion. Only search in the files under git control.

Version 0, edited 3 years ago by Frédéric Chapoton (next)

comment:7 in reply to:  6 Changed 3 years ago by John Palmieri

Replying to chapoton:

Ok, then we will see later when we upgrade matplotlib.

By the way, do you know "git grep" ? Useful tool, in my opinion. Only search in the files under git control.

No, I didn't know, thanks for pointing it out.

Last edited 3 years ago by John Palmieri (previous) (diff)

comment:8 Changed 3 years ago by Samuel Lelièvre

See also #28012.

comment:9 Changed 3 years ago by Volker Braun

Branch: u/jhpalmieri/cmdline-py38712a2fab3fbab9944a4a08fc670515bd0b39ff2
Resolution: fixed
Status: positive_reviewclosed

comment:10 Changed 3 years ago by Erik Bray

Milestone: sage-8.8sage-8.9

Not in Sage 8.8. Let's please to try keep tickets' milestones related to the release in which we actually intend to include them, and in particular the release in which they were actually included, especially when closing tickets.

Note: See TracTickets for help on using tickets.