Opened 9 years ago
Closed 9 years ago
#16146 closed enhancement (fixed)
FiniteStateMachine._latex_: initial states taken into account
Reported by: | cheuberg | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | sage-6.2 |
Component: | combinatorics | Keywords: | finite_state_machine |
Cc: | skropf, dkrenn | Merged in: | |
Authors: | Clemens Heuberger | Reviewers: | Daniel Krenn |
Report Upstream: | N/A | Work issues: | |
Branch: | fe9b380 (Commits, GitHub, GitLab) | Commit: | fe9b3803bf0294c05014f76cc3bc8b5bd1377358 |
Dependencies: | #16200 | Stopgaps: |
Description
Previously, initial states were not taken into account.
As the output is in tikz, the parameter choosing the direction of the arrow indicating the initial state follows tikz' notations. Thus, initial_where
can be set to one of
"below"
, "above"
, "left"
, "right"
.
Change History (10)
comment:1 Changed 9 years ago by
Status: | new → needs_review |
---|
comment:2 Changed 9 years ago by
Commit: | 50397ef8c38651a3de7e8485fa66fd0905e06f21 → a63c975c05c5855665942f4a950632e5925ad3d8 |
---|
comment:3 Changed 9 years ago by
Branch: | u/cheuberg/fsm/latex-initial-state → u/dkrenn/fsm/latex-initial-state |
---|
comment:4 Changed 9 years ago by
Commit: | a63c975c05c5855665942f4a950632e5925ad3d8 → ef79bba711b33471c4cfea3b0f518b511bf1f70e |
---|---|
Reviewers: | → Daniel Krenn |
Very minor change.
I made another commit to make all print-statements comaptible with Python 3. Please review those and if they are fine, feel free to set this ticket to positive_review
New commits:
002c57b | added 'print' in doctests (F._latex_) to get nicer output
|
ef79bba | python3: "print ..." to print(...)
|
comment:5 Changed 9 years ago by
Status: | needs_review → needs_work |
---|
In its current form, this will create a merge conflict with #16200. Pending resolution, I set the status back to needs_work.
comment:6 Changed 9 years ago by
Branch: | u/dkrenn/fsm/latex-initial-state → u/cheuberg/fsm/latex-initial-state |
---|---|
Commit: | ef79bba711b33471c4cfea3b0f518b511bf1f70e → 002c57bd10785e90549b987ca5f6df2d47053432 |
comment:7 Changed 9 years ago by
Commit: | 002c57bd10785e90549b987ca5f6df2d47053432 → 98328da0551d82a774c098d1065fec3873bdc109 |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
4d389d9 | FiniteStateMachine._latex_: typeset state labels in math mode (#16200)
|
89c10cc | Merge remote-tracking branch 'aau/fsm/latex-state-label' #16200 into fsm/latex-initial-state-new
|
98328da | replace print(F._latex_()) by print latex(F) (three times)
|
comment:8 Changed 9 years ago by
Dependencies: | → #16200 |
---|---|
Status: | needs_work → needs_review |
#16120 is now merged.
comment:9 Changed 9 years ago by
Branch: | u/cheuberg/fsm/latex-initial-state → u/dkrenn/fsm/latex-initial-state_b |
---|---|
Commit: | 98328da0551d82a774c098d1065fec3873bdc109 → fe9b3803bf0294c05014f76cc3bc8b5bd1377358 |
Status: | needs_review → positive_review |
Marked three doctests as indirect. Now everything is fine.
New commits:
fe9b380 | marked 3 doctests as indirect
|
comment:10 Changed 9 years ago by
Branch: | u/dkrenn/fsm/latex-initial-state_b → fe9b3803bf0294c05014f76cc3bc8b5bd1377358 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Branch pushed to git repo; I updated commit sha1. New commits:
Merge tag '6.2.beta8' into t/16146/fsm/latex-initial-state