#28398 closed enhancement (fixed)
_richcmp_ for quaternion algebra elements
Reported by: | chapoton | Owned by: | chapoton |
---|---|---|---|
Priority: | major | Milestone: | sage-8.9 |
Component: | algebra | Keywords: | |
Cc: | tscrim, chapoton | Merged in: | |
Authors: | Frédéric Chapoton | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | c8425cb (Commits, GitHub, GitLab) | Commit: | |
Dependencies: | Stopgaps: |
Description
instead of the old-style _cmp_
Change History (8)
comment:1 Changed 3 years ago by
- Branch set to u/chapoton/28398
- Commit set to c8425cb310b3133b0b51bbdb286fd7847f0d025c
- Status changed from new to needs_review
comment:3 Changed 3 years ago by
- Reviewers set to Travis Scrimshaw
- Status changed from needs_review to positive_review
LGTM.
comment:4 Changed 3 years ago by
- Branch changed from u/chapoton/28398 to c8425cb310b3133b0b51bbdb286fd7847f0d025c
- Resolution set to fixed
- Status changed from positive_review to closed
comment:5 Changed 2 years ago by
- Commit c8425cb310b3133b0b51bbdb286fd7847f0d025c deleted
- Owner changed from (none) to tscrim
This breaks quaternion algebra comparisons quite badly:
sage: B.<i,j,k> = QuaternionAlgebra(6) sage: B(1) == B(-1) False sage: B(1) != B(-1) True
Version 0, edited 2 years ago
by
(next)
comment:6 Changed 2 years ago by
- Cc chapoton added
comment:7 Changed 2 years ago by
- Owner changed from tscrim to chapoton
comment:8 Changed 2 years ago by
Note: See
TracTickets for help on using
tickets.
New commits:
richcmp for quaternion algebras elements