#16580: FiniteStateMachine: cleanup code
-------------------------------------+-------------------------------------
Reporter: cheuberg | Owner:
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-6.3
Component: finite state | Resolution:
machines | Merged in:
Keywords: | Reviewers: Daniel Krenn
finite_state_machine | Work issues:
Authors: Clemens Heuberger | Commit:
Report Upstream: N/A | 102be36393c29989a9c8a2346c1f087e5c03ad04
Branch: u/cheuberg/fsm | Stopgaps:
/code-cleanup-post-16067 |
Dependencies: #16557 |
-------------------------------------+-------------------------------------
Changes (by cheuberg):
* status: needs_work => needs_review
* dependencies: => #16557
Comment:
Thank you for your comments, here and on sage-devel.
So I understand that the result of {{{0 < 'I'}}} is intentionally random.
For the particular two sometimes failing doctests (I now could reproduce
the failure by calling {{{sage -t}}} via a Makefile), the solution seems
to be actually quite simple: In the sample transducer, I renamed the state
{{{'I'}}} to {{{4}}}, now we are sorting integers and that should be well-
defined.
This required merging of #16557 in order to avoid a merge conflict.
Trac's automerge is still unhappy, but I cannot see the reason.
--
Ticket URL: <http://trac.sagemath.org/ticket/16580#comment:15>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
