Opened 6 years ago

Closed 20 months ago

Last modified 16 months ago

#16081 closed enhancement (wontfix)

Python3 : Test ticket for range()

Reported by: wluebbe Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: python3 Keywords: python3
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #16457 Stopgaps:

Description (last modified by chapoton)

This ticket is used for doctesting the Python 3 compatibility of range(). It is not a real ticket and should not be merged.

Python 3 preparation: "xrange()" is gone and "range()" becomes an iterator

The tool 2to3 renames xrange() to range() and wraps existing range() usages with list().
But the code has to depend on the Python version!

There are 293 effected modules.

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

Change History (27)

comment:1 Changed 6 years ago by wluebbe

  • Type changed from defect to enhancement

comment:2 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:3 Changed 5 years ago by aapitzsch

  • Dependencies set to #16457

I moved most of the renaming part to #16457.

comment:4 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:5 Changed 3 years ago by jdemeyer

  • Component changed from distribution to python3

comment:6 Changed 3 years ago by chapoton

see #21375 for a step in this direction

comment:7 Changed 3 years ago by chapoton

see #21460 for another small step

comment:8 Changed 3 years ago by chapoton

  • Milestone changed from sage-6.4 to sage-7.4

comment:9 Changed 3 years ago by chapoton

see #21677 for another step

comment:10 Changed 3 years ago by chapoton

  • Milestone changed from sage-7.4 to sage-7.6

comment:11 Changed 3 years ago by chapoton

  • Cc jdemeyer tscrim jmantysalo added
  • Milestone changed from sage-7.6 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review
Last edited 3 years ago by chapoton (previous) (diff)

comment:12 Changed 3 years ago by chapoton

  • Cc jdemeyer tscrim jmantysalo removed
  • Milestone changed from sage-duplicate/invalid/wontfix to sage-7.6
  • Status changed from needs_review to needs_work

sorry, I did a wrong move here, this is probably not yet ready to be closed

comment:13 Changed 3 years ago by chapoton

  • Branch set to u/chapoton/test_branch_for_range
  • Commit set to 505ce41e40a6006e445c32e12f5a4be6b85fc252

here is a branch, NOT TO BE MERGED, to let the patchbots run and detect remaining problems


New commits:

505ce41py3 : test branch for range !!! DO NOT MERGE !!!

comment:14 Changed 3 years ago by chapoton

  • Milestone changed from sage-7.6 to sage-duplicate/invalid/wontfix

comment:15 Changed 3 years ago by chapoton

see #22521 for one step in this direction

comment:16 Changed 3 years ago by chapoton

another step in #22734

comment:17 Changed 3 years ago by chapoton

  • Description modified (diff)
  • Summary changed from Python 3 preparation: "xrange()" is gone and "range()" becomes an iterator to Python3 : Test ticket for range()

comment:18 Changed 3 years ago by chapoton

  • Description modified (diff)

comment:19 Changed 3 years ago by chapoton

next iteration in #22779

comment:20 Changed 3 years ago by chapoton

next step in #22816

comment:21 Changed 2 years ago by chapoton

next step in #22972

comment:22 Changed 2 years ago by etn40ff

Matrices here #23107

comment:23 Changed 2 years ago by chapoton

I have just run the patchbot. There is still a lot of things to be done here.

comment:24 Changed 2 years ago by chapoton

next in #24249

comment:25 Changed 21 months ago by chapoton

next step in #24569

comment:26 Changed 20 months ago by embray

  • Resolution set to wontfix
  • Status changed from needs_work to closed

Going ahead and closing (unless you disagree) since it's not a real change to commit. There aren't too many range-related issues remaining (and those that do remain are mostly in classes/functions that should accept range objects as arguments but currently don't).

comment:27 Changed 16 months ago by chapoton

  • Branch u/chapoton/test_branch_for_range deleted
  • Commit 505ce41e40a6006e445c32e12f5a4be6b85fc252 deleted
Note: See TracTickets for help on using tickets.