Opened 4 years ago

Closed 4 years ago

#21802 closed enhancement (fixed)

py3 cmp in some cython files (morphism.pyx, etc)

Reported by: chapoton Owned by:
Priority: major Milestone: sage-7.5
Component: python3 Keywords:
Cc: jdemeyer, jmantysalo, embray, tscrim Merged in:
Authors: Frédéric Chapoton Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: d74a374 (Commits) Commit: d74a374970936ba680ebcee22ba1da3cf1110593
Dependencies: #21779 Stopgaps:

Description

as another tiny step in the huge cmp problem on the way to py3

Change History (16)

comment:1 Changed 4 years ago by chapoton

  • Branch set to u/chapoton/21802
  • Commit set to 056c995ee8efaa90c97a2afb818f09bc7d622b7b
  • Dependencies set to #21779

New commits:

​056c995more work on cmp() in cython files

comment:2 Changed 4 years ago by jdemeyer

Why the dependency on #21779?

comment:3 Changed 4 years ago by chapoton

because I am using the brand new richcmp_not_equal

comment:4 Changed 4 years ago by jdemeyer

In that case, the branch should actually be based on top of #21779.

comment:5 Changed 4 years ago by git

  • Commit changed from 056c995ee8efaa90c97a2afb818f09bc7d622b7b to 19ae3c90f950e2682a49d6c7e926499e50fc6173

Branch pushed to git repo; I updated commit sha1. New commits:

​04897a1Merge branch 'u/chapoton/21802' in 7.5.b2
​19ae3c9trac 21802 correcting the code

comment:6 Changed 4 years ago by git

  • Commit changed from 19ae3c90f950e2682a49d6c7e926499e50fc6173 to 1c0619885bb378166353d3a79cfb8bef621e8e3e

Branch pushed to git repo; I updated commit sha1. New commits:

​1c06198trac 21802 fixing cmp (to richcmp) in elliptic curve isogenies

comment:7 Changed 4 years ago by chapoton

there remains some problems with toric morphisms

comment:8 Changed 4 years ago by jdemeyer

As in the other ticket, you should use 

left = <FastDoubleFunc?>self
right = <FastDoubleFunc?>other

instead of 

left, right = self, other

to check that the type is correct.

comment:9 Changed 4 years ago by git

  • Commit changed from 1c0619885bb378166353d3a79cfb8bef621e8e3e to a0d7e02ba9b36966d28900b9d3dfdb61c2d94120

Branch pushed to git repo; I updated commit sha1. New commits:

​a0d7e02trac 21802 fixing toric morphisms cmp

comment:10 Changed 4 years ago by git

  • Commit changed from a0d7e02ba9b36966d28900b9d3dfdb61c2d94120 to d74a374970936ba680ebcee22ba1da3cf1110593

Branch pushed to git repo; I updated commit sha1. New commits:

​d74a374trac 21802 reviewer's comment

comment:11 Changed 4 years ago by chapoton

done, thanks

comment:12 Changed 4 years ago by chapoton

  • Status changed from new to needs_review

bot is green, please review

comment:13 Changed 4 years ago by chapoton

  • Cc jdemeyer jmantysalo embray tscrim added

bot is green, please review

comment:14 Changed 4 years ago by chapoton

*please*, could somebody review this simple ticket ?

comment:15 Changed 4 years ago by tscrim

  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

In lieu of any additional comments by Jeroen, I give a positive review.

comment:16 Changed 4 years ago by vbraun

  • Branch changed from u/chapoton/21802 to d74a374970936ba680ebcee22ba1da3cf1110593
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.