Skip to content

Commit 604f7d1

Browse files
committed
use more strict test
1 parent 1d7ca02 commit 604f7d1

1 file changed

Lines changed: 12 additions & 5 deletions

File tree

sklearn/linear_model/tests/test_ridge.py

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -855,19 +855,26 @@ def test_ridge_regression_check_arguments_validity(return_intercept,
855855
X = rng.rand(1000, 3)
856856
true_coefs = [1, 2, 0.1]
857857
y = np.dot(X, true_coefs)
858+
intercept = 0.
859+
if return_intercept:
860+
intercept = 10000.
861+
y += intercept
858862
X_testing = arr_type(X)
859863

860-
target = ridge_regression(X_testing, y, 1,
864+
alpha, atol, tol = 1e-3, 1e-4, 1e-6
865+
target = ridge_regression(X_testing, y, alpha=alpha,
861866
solver=solver,
862867
sample_weight=sample_weight,
863-
return_intercept=return_intercept
868+
return_intercept=return_intercept,
869+
tol=tol,
864870
)
871+
865872
if return_intercept:
866873
coef, intercept = target
867-
assert_allclose(coef, true_coefs, rtol=0, atol=0.03)
868-
assert_allclose(intercept, 0, rtol=0, atol=0.03)
874+
assert_allclose(coef, true_coefs, rtol=0, atol=atol)
875+
assert_allclose(intercept, intercept, rtol=0, atol=atol)
869876
else:
870-
assert_allclose(target, true_coefs, rtol=0, atol=0.03)
877+
assert_allclose(target, true_coefs, rtol=0, atol=atol)
871878

872879

873880
def test_ridge_regression_warns_with_return_intercept():

0 commit comments

Comments
 (0)