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.

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

Python 3 preparation: Some functions (ifilter, imap, izip) are removed from itertools #16070

Closed
wluebbe mannequin opened this issue Apr 7, 2014 · 6 comments
Closed

Comments

@wluebbe
Copy link
Mannequin

wluebbe mannequin commented Apr 7, 2014

The tool 2to3 removes the imports.

But the code has to depend on the Python version!

An alternative approach is to add from future_builtins import filter and to check where a wrapping with list() is required.

There are 13 effected modules.

This ticket is tracked as a dependency of meta-ticket ticket:16052.

Component: python3

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

@wluebbe wluebbe mannequin added this to the sage-6.2 milestone Apr 7, 2014
@wluebbe wluebbe mannequin added c: distribution labels Apr 7, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@wluebbe
Copy link
Mannequin Author

wluebbe mannequin commented May 22, 2014

Dependencies: #16071

@wluebbe
Copy link
Mannequin Author

wluebbe mannequin commented May 22, 2014

comment:2

Added dependency: The imports may only be removed when their usages have been removed in #16071.

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@fchapoton
Copy link
Contributor

comment:5

for imap, see #21364

@fchapoton
Copy link
Contributor

comment:6

There is no longer any ifilter to take care of.

For imap, see tickets #21364 and #21373

and for izip, see #21374

@fchapoton
Copy link
Contributor

comment:7

let us close this as duplicate

@fchapoton fchapoton removed this from the sage-6.4 milestone Aug 30, 2016
@fchapoton
Copy link
Contributor

Changed dependencies from #16071 to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants