Skip to content

Commit

Permalink
Remove inadvertently included debugging code
Browse files Browse the repository at this point in the history
  • Loading branch information
kshedden committed May 5, 2024
1 parent d913ae8 commit baf7be1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion statsmodels/multivariate/factor.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,6 @@ def __init__(self, endog=None, n_factor=1, corr=None, method='pa',
if cfa is not None and method == "ml":
self.n_factor = cfa.shape[0] // self.k_endog

Check warning on line 151 in statsmodels/multivariate/factor.py

View check run for this annotation

Codecov / codecov/patch

statsmodels/multivariate/factor.py#L151

Added line #L151 was not covered by tests
if cfa.shape[0] != self.n_factor * self.k_endog:
1/0
raise ValueError("Leading dimension of 'cfa' must be divisible by number of variables")
self.cfa = cfa

Check warning on line 154 in statsmodels/multivariate/factor.py

View check run for this annotation

Codecov / codecov/patch

statsmodels/multivariate/factor.py#L153-L154

Added lines #L153 - L154 were not covered by tests
elif cfa is not None:
Expand Down
1 change: 1 addition & 0 deletions statsmodels/multivariate/tests/test_factor.py
Original file line number Diff line number Diff line change
Expand Up @@ -263,6 +263,7 @@ def test_cfa_builder():
mod = Factor(X, 2, method="ml", cfa=cfa4)
rslt = mod.fit()
ld = rslt.loadings
assert_allclose((np.abs(ld) > 1e-5).sum(), 5)

Check warning on line 266 in statsmodels/multivariate/tests/test_factor.py

View check run for this annotation

Codecov / codecov/patch

statsmodels/multivariate/tests/test_factor.py#L261-L266

Added lines #L261 - L266 were not covered by tests


@pytest.mark.skipif(missing_matplotlib, reason='matplotlib not available')
Expand Down

0 comments on commit baf7be1

Please sign in to comment.