Opened 9 years ago
Closed 9 years ago
#12475 closed defect (fixed)
Define WordMorphism.__ne__
Reported by: | tjolivet | Owned by: | sage-combinat |
---|---|---|---|
Priority: | major | Milestone: | sage-5.0 |
Component: | combinatorics | Keywords: | |
Cc: | slabbe | Merged in: | sage-5.0.beta5 |
Authors: | Sébastien Labbé | Reviewers: | Timo Jolivet |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Solve the following problem:
sage: s = WordMorphism('1->121,2->131,3->4,4->1') sage: s == s.reversal() True sage: s != s.reversal() True
which is due to:
sage: s.__ne__? Object `s.__ne__` not found.
Not much more than this should be added:
def __ne__(self, other): return not self.__eq__(other)
Attachments (1)
Change History (4)
Changed 9 years ago by
comment:1 Changed 9 years ago by
- Status changed from new to needs_review
comment:2 Changed 9 years ago by
- Reviewers set to Timo Jolivet
- Status changed from needs_review to positive_review
comment:3 Changed 9 years ago by
- Merged in set to sage-5.0.beta5
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
The patch applies correctly on my system and fixes the reported problem: positive review.
I couldn't figure out where the PatchBot? fails (yellow icon as of now).