slinderman / recurrent-slds

Recurrent Switching Linear Dynamical Systems
MIT License
97 stars 23 forks source link

transposing trans matrices #1

Closed slinderman closed 7 years ago

slinderman commented 7 years ago

Is this needed? https://github.com/slinderman/recurrent-slds/blob/master/rslds/inhmm.py#L216 What if we just call https://github.com/mattjj/pyhsmm/blob/3ba405eca0a8d00f222d8470dc7a850281eb5085/pyhsmm/internals/hmm_states.py#L422 instead? We should profile but I think these manual array constructions are relatively expensive.

slinderman commented 7 years ago

No, it was not needed, and in fact, it was causing a pretty significant slowdown.