#31704 closed defect (fixed)

Remove Python 2 compatibility from manifolds

Reported by: Michael Jung Owned by:
Priority: major Milestone: sage-9.4
Component: manifolds Keywords:
Cc: Eric Gourgoulhon, Travis Scrimshaw Merged in:
Authors: Eric Gourgoulhon Reviewers: Michael Jung
Report Upstream: N/A Work issues:
Branch: b027f2d (Commits, GitHub, GitLab) Commit: b027f2d1a4bb09b93e72c6c97f2ab739ddafd70c
Dependencies: Stopgaps:

Status badges

Description

Currently, the manifolds code has still things like

__nonzero__ = __bool__  # For Python2 compatibility

which should be removed by now.

Change History (6)

comment:1 Changed 20 months ago by Eric Gourgoulhon

Authors: Eric Gourgoulhon
Branch: public/manifolds/remove_python2_compatibility-31704
Commit: 0aefe529a51100b5870194aeb767e3cc7a2cade5
Status: newneeds_review
Type: taskdefect

Thanks for catching this. Here we go...


New commits:

0aefe52Remove __nonzero__ from manifolds

comment:2 Changed 20 months ago by Michael Jung

Reviewers: Michael Jung
Status: needs_reviewpositive_review

LGTM. Morally green patchbot. The cause of that pyflakes error is in line 1507 of free_module_tensor.py:

if basis is None: basis = self._fmodule._def_basis

Should we fix this in another ticket just to get rid of this annoying error whenever we play around with free_module_tensor.py...?

comment:3 Changed 20 months ago by git

Commit: 0aefe529a51100b5870194aeb767e3cc7a2cade5b027f2d1a4bb09b93e72c6c97f2ab739ddafd70c
Status: positive_reviewneeds_review

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

b027f2dCorrect pycodestyle error

comment:4 in reply to:  2 Changed 20 months ago by Eric Gourgoulhon

Replying to gh-mjungmath:

LGTM. Morally green patchbot. The cause of that pyflakes error is in line 1507 of free_module_tensor.py:

if basis is None: basis = self._fmodule._def_basis

Should we fix this in another ticket just to get rid of this annoying error whenever we play around with free_module_tensor.py...?

You're right; let's fix it here (cf. the latest commit).

comment:5 Changed 20 months ago by Michael Jung

Status: needs_reviewpositive_review

Morally green. LGTM.

comment:6 Changed 18 months ago by Volker Braun

Branch: public/manifolds/remove_python2_compatibility-31704b027f2d1a4bb09b93e72c6c97f2ab739ddafd70c
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.