Opened 8 years ago

Closed 8 years ago

#16355 closed defect (fixed)

FiniteStateMachine: Corrected adjacency matrix

Reported by: skropf Owned by:
Priority: minor Milestone: sage-6.3
Component: combinatorics Keywords: finite_state_machine
Cc: dkrenn, cheuberg Merged in:
Authors: Sara Kropf Reviewers: Clemens Heuberger
Report Upstream: N/A Work issues:
Branch: d1a3180 (Commits, GitHub, GitLab) Commit: d1a3180ffa71636d88110d012a226d4adce10ab4
Dependencies: Stopgaps:

Status badges


Until now, the default entry for the adjacency matrix of the underlying graph was not correctly computed for finite state machines with transitions whose output has length not equal 1.

This is now corrected. Also an example to demonstrate this is added to the documentation.

Change History (4)

comment:1 Changed 8 years ago by skropf

  • Status changed from new to needs_review

comment:2 Changed 8 years ago by git

  • Commit changed from 2d56c5093188294e923bd518780fe20e92a55be6 to d1a3180ffa71636d88110d012a226d4adce10ab4

Branch pushed to git repo; I updated commit sha1. New commits:

d1a3180FiniteStateMachine.adjacency_matrix: additional example in documentation

comment:3 Changed 8 years ago by cheuberg

  • Reviewers set to Clemens Heuberger
  • Status changed from needs_review to positive_review

reviewed change; merges cleanly with relevant branches in 6.3.beta.1, doctests pass, documentation ok.

comment:4 Changed 8 years ago by vbraun

  • Branch changed from u/skropf/fsm/adjacency_matrix to d1a3180ffa71636d88110d012a226d4adce10ab4
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.