Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#18473 closed enhancement (fixed)

Python 3: The semantic of map() function is changed (part2)

Reported by: aapitzsch Owned by:
Priority: major Milestone: sage-6.8
Component: misc Keywords: python3
Cc: jdemeyer Merged in:
Authors: André Apitzsch Reviewers: Wilfried Luebbe
Report Upstream: N/A Work issues:
Branch: bb90e06 (Commits) Commit:
Dependencies: Stopgaps:

Description

This ticket is part of #16073 to simplify review.

In Py2 map() returns a list, while in Py3 map() returns an iterator.

Change History (4)

comment:1 Changed 5 years ago by aapitzsch

  • Cc jdemeyer added
  • Status changed from new to needs_review

comment:2 Changed 5 years ago by wluebbe

  • Reviewers set to Wilfried Lübbe
  • Status changed from needs_review to positive_review

All test passed and the changes look good to me.

There are still some overlooked maps in the files of src/sage/combinat/*. But I will create a followup ticket/patch to address those (so that the effort to review the current patch is not wasted).

comment:3 Changed 5 years ago by vbraun

  • Branch changed from u/aapitzsch/map_p2 to bb90e06a42b11cdc1058bc3e1e289546351a93ce
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:4 Changed 5 years ago by jdemeyer

  • Commit bb90e06a42b11cdc1058bc3e1e289546351a93ce deleted
  • Reviewers changed from Wilfried Lübbe to Wilfried Luebbe
Note: See TracTickets for help on using tickets.