Opened 6 years ago

Closed 4 years ago

#19170 closed enhancement (wontfix)

Comparison modifications in algebras/

Reported by: vdelecroix Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: python3 Keywords:
Cc: Merged in:
Authors: Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: u/vdelecroix/19170 (Commits, GitHub, GitLab) Commit: 861cce24b8301821e3291ae53b22dccc094d9ebc
Dependencies: Stopgaps:

Status badges

Description

Modify some comparisons in sage.algebras to use the _cmp_ and _richcmp_ involving the coercion framework.

This ticket was initially part of #18305.

Change History (11)

comment:1 Changed 6 years ago by vdelecroix

  • Branch set to u/vdelecroix/19170
  • Commit set to 861cce24b8301821e3291ae53b22dccc094d9ebc
  • Dependencies set to #18305
  • Status changed from new to needs_review

New commits:

a9283cbTrac 18305: py_rich_to_bool(_sgn) and operators
e8b8182Trac 18305: _richcmp_ for indexed monoids and groups
950f3ecTrac 18305: _richcmp_ for Newton polygons
490de42Trac 18305: comments about Python comparisons
3f6ea84Trac 18305: rephrased sentence in doc
4949e17Trac 19170: use _cmp_ for ideals of finite dimensional algebras
861cce2Trac 19170: _richcmp_ for `finite dimensional algebra elements

comment:2 Changed 6 years ago by kdilks

  • Milestone changed from sage-6.9 to sage-6.10

Merge conflicts.

comment:3 Changed 5 years ago by chapoton

  • Status changed from needs_review to needs_work

does not apply

comment:4 Changed 5 years ago by chapoton

  • Component changed from coercion to python3
  • Milestone changed from sage-6.10 to sage-8.1

comment:5 Changed 5 years ago by chapoton

  • Dependencies #18305 deleted

comment:6 Changed 4 years ago by chapoton

  • Status changed from needs_work to needs_info

This is probably obsolete.

comment:7 Changed 4 years ago by chapoton

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

comment:8 Changed 4 years ago by chapoton

  • Status changed from needs_info to positive_review

Ok, let us consider that this one also is way outdated.

comment:9 Changed 4 years ago by tscrim

  • Authors Vincent Delecroix deleted
  • Reviewers set to Frédéric Chapoton

There is not a def __cmp__ in sage/algebras/ from grepping, so this has been taken care of.

comment:10 Changed 4 years ago by chapoton

Yes, I wonder by who. But there remains to handle comparison of "ideals" in rings. I will work on that once the current py3 tickets have been included in sage.

comment:11 Changed 4 years ago by embray

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.