Opened 2 years ago

Closed 23 months ago

#30715 closed enhancement (fixed)

Remove a few .all imports

Reported by: Tobias Diez Owned by:
Priority: minor Milestone: sage-9.3
Component: refactoring Keywords:
Cc: Matthias Köppe Merged in:
Authors: Tobias Diez Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 875ba9e (Commits, GitHub, GitLab) Commit: 875ba9eed8e7c0c95c50a3a51b7102a9ab2bbc53
Dependencies: Stopgaps:

Status badges


Remove a few imports, so that the whole module xyz.all doesn't need to be loaded. That's not so important for the normal usage of sage, as there sage.all loads these other all imports as well. But this gets more important when part of sage is used as a library, and for the upcoming modularization.

Also two unused imports are removed. The other changes are cosmetic (from ... import).

Change History (5)

comment:1 Changed 2 years ago by Tobias Diez

Status: newneeds_review

comment:2 Changed 2 years ago by Matthias Köppe

Reviewers: Matthias Koeppe
Status: needs_reviewpositive_review

comment:3 Changed 2 years ago by Tobias Diez

Thanks for the review!

comment:4 Changed 2 years ago by Matthias Köppe

Milestone: sage-9.2sage-9.3

comment:5 Changed 23 months ago by Volker Braun

Branch: public/refactoring/allImports875ba9eed8e7c0c95c50a3a51b7102a9ab2bbc53
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.