Skip to content

Commit 3f4366b

Browse files
authored
Merge pull request #441 from colecitrenbaum/main
Off by one in input driven HMM
2 parents 11c1c9f + 14cb0ce commit 3f4366b

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

dynamax/hidden_markov_model/models/abstractions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ def collect_suff_stats(self,
295295
PyTree of sufficient statistics for updating the transition distribution
296296
297297
"""
298-
return posterior.trans_probs, pytree_slice(inputs, slice(1, None))
298+
return posterior.trans_probs, inputs
299299

300300
def initialize_m_step_state(self, params: ParameterSet, props:PropertySet) -> Any:
301301
"""Initialize any required state for the M step.

pyproject.toml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ test = [
7070
"coverage",
7171
"pytest>=3.9",
7272
"pytest-cov",
73-
"interrogate>=1.5.0"
73+
"interrogate>=1.5.0",
74+
"ipython"
7475
]
7576

7677
dev = [
@@ -91,7 +92,8 @@ dev = [
9192
"coverage",
9293
"pytest>=3.9",
9394
"pytest-cov",
94-
"interrogate>=1.5.0"
95+
"interrogate>=1.5.0",
96+
"ipython"
9597
]
9698

9799
[tool.setuptools.packages.find]

0 commit comments

Comments
 (0)