sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.38k stars 469 forks source link

FiniteStateMachine.transposition(): multi-letter input not handeled correctly. #19453

Closed cheuberg closed 8 years ago

cheuberg commented 8 years ago

The following behaviour is not intended:

sage: A = Automaton([(0, 1, [1, 0])],
....:     initial_states=[0],
....:     final_states=[1])
sage: A([1, 0])
True
sage: A.transposition()([0, 1])
False

CC: @sagetrac-skropf @dkrenn

Component: finite state machines

Author: Clemens Heuberger

Branch/Commit: 3a75d03

Reviewer: Sara Kropf

Issue created by migration from https://trac.sagemath.org/ticket/19453

cheuberg commented 8 years ago

Branch: u/cheuberg/fsm/transposition

cheuberg commented 8 years ago

Commit: 3a75d03

cheuberg commented 8 years ago

New commits:

7ab7b37Trac #19453: FiniteStateMachine.transposition(): multi-letter input not handeled correctly
3a75d03Trac #19453: also transpose output
c50b3d32-6cb1-4b90-a060-6a332e54ef6a commented 8 years ago

Reviewer: Sara Kropf

c50b3d32-6cb1-4b90-a060-6a332e54ef6a commented 8 years ago
comment:3

This is fine. Positive review.

vbraun commented 8 years ago

Changed branch from u/cheuberg/fsm/transposition to 3a75d03