Opened 8 years ago
Closed 8 years ago
#15849 closed enhancement (fixed)
Default ordering of States and Transitions in FiniteStateMachine
Reported by: | cheuberg | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | sage-6.2 |
Component: | combinatorics | Keywords: | finite_state_machine |
Cc: | Merged in: | ||
Authors: | Clemens Heuberger, Daniel Krenn | Reviewers: | Daniel Krenn |
Report Upstream: | N/A | Work issues: | |
Branch: | 9af0b91 (Commits, GitHub, GitLab) | Commit: | 9af0b91992bf0ff267a2c8c684b1f79b456f4a1e |
Dependencies: | Stopgaps: |
Description (last modified by )
Provided methods
sage.combinat.finite_state_machine.FSMState.__lt__
and
sage.combinat.finite_state_machine.FSMTransition.__lt__
in order to have default ordering. For instance, this helps enforcing a unique output in
doctests.
Change History (6)
comment:1 Changed 8 years ago by
- Type changed from PLEASE CHANGE to enhancement
comment:2 Changed 8 years ago by
- Branch set to u/cheuberg/fsm/default-order-states-transitions
- Cc dkrenn added
- Commit set to 9af0b91992bf0ff267a2c8c684b1f79b456f4a1e
- Description modified (diff)
- Status changed from new to needs_review
comment:3 Changed 8 years ago by
- Cc dkrenn removed
- Reviewers set to Daniel Krenn
comment:4 Changed 8 years ago by
- Status changed from needs_review to positive_review
comment:5 Changed 8 years ago by
Corrected spacings (PEP8)
code looks fine, doctests pass, docu checked, coverage checked
comment:6 Changed 8 years ago by
- Branch changed from u/cheuberg/fsm/default-order-states-transitions to 9af0b91992bf0ff267a2c8c684b1f79b456f4a1e
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
implemented FSMTransition.__lt__ and FSMState.__lt__
rewrote docstring of .__lt__ in FSMState/FSMTransition; corrected spacings
added one spaceing (PEP8)
Whitespace correction in FSMTransition.__lt__