You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
scipy is currently a required dependency, but it's only used by SignRestriction in identification.py (specifically scipy.stats.special_ortho_group for generating random rotation matrices).
Proposed Solution
Move scipy from dependencies to [project.optional-dependencies]:
Obsolete by evolution. scipy is no longer confined to SignRestriction: it is imported at module level by the conjugate estimator (src/impulso/_conjugate.py:32-33 — multigammaln, invwishart), its hyperparameter sampler (src/impulso/_conjugate_sampler.py:31-33 — optimize, special.expit, stats.gamma), and the pandemic-break hyperpriors (src/impulso/conjugate_volatility.py:28), with scipy.stats.chi2 also used in fitted.py:256 and identified.py:629. Making it optional would push ConjugateVAR behind an extra for no meaningful install-footprint win. Closing.
Activity
refactor: extract MA recursion + lift time-dim guard to base (closes #76
refactor: extract MA recursion + lift time-dim guard to base (closes #76
thomaspinder commentedon Jul 29, 2026
Obsolete by evolution. scipy is no longer confined to
SignRestriction: it is imported at module level by the conjugate estimator (src/impulso/_conjugate.py:32-33—multigammaln,invwishart), its hyperparameter sampler (src/impulso/_conjugate_sampler.py:31-33—optimize,special.expit,stats.gamma), and the pandemic-break hyperpriors (src/impulso/conjugate_volatility.py:28), withscipy.stats.chi2also used infitted.py:256andidentified.py:629. Making it optional would pushConjugateVARbehind an extra for no meaningful install-footprint win. Closing.