Ticket #4189 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[with patch; positive review] hmm documentation buglet

Reported by: mabshoff Owned by: was
Priority: major Milestone: sage-3.1.3
Component: documentation Keywords:
Cc: Work issues:
Report Upstream: Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

mapb reported in  http://groups.google.com/group/sage-devel/t/8109b386f0e94251 

The fourth argument in the following routine is called 
"emission_symbols", while the INPUTS section reports "emission_state". 
hmm.DiscreteHiddenMarkovModel(A, B, pi=None, emission_symbols=None, 
name=None, normalize=True) 
n 
    INPUTS: 
        A  -- square matrix of doubles; the state change probabilities 
        B  -- matrix of doubles; emission probabilities 
        pi -- list of floats; probabilities for each initial state 
        emission_state -- list of B.ncols() symbols (just used for 
printing) 
        name -- (optional) name of the model 
        normalize -- (optional; default=True) whether or not to 
normalize 
                     the model so the probabilities add to 1

Attachments

trac-4189.patch Download (964 bytes) - added by was 5 years ago.

Change History

comment:1 Changed 5 years ago by was

  • Summary changed from hmm documentation buglet to [with patch; needs review] hmm documentation buglet

Changed 5 years ago by was

comment:2 Changed 5 years ago by mabshoff

  • Summary changed from [with patch; needs review] hmm documentation buglet to [with patch; positive review] hmm documentation buglet

Looks good to me, positive review.

Cheers,

Michael

comment:3 Changed 5 years ago by mabshoff

  • Status changed from new to closed
  • Resolution set to fixed

Merged in Sage 3.1.3.alpha2

Note: See TracTickets for help on using tickets.