Commite4b0849

authored

FIX Avoid fitting a pipeline without steps (#31723)

1 parent9b7a86f commite4b0849Copy full SHA for e4b0849

File tree

+12

-1

lines changed

+12

-1

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -320,6 +320,8 @@ def set_params(self, **kwargs):`
`320`	`320`	`returnself`
`321`	`321`
`322`	`322`	`def_validate_steps(self):`
	`323`	`+ifnotself.steps:`
	`324`	`+raiseValueError("The pipeline is empty. Please add steps.")`
`323`	`325`	`names,estimators=zip(*self.steps)`
`324`	`326`
`325`	`327`	`# validate names`
`@@ -1289,7 +1291,6 @@ def __sklearn_is_fitted__(self):`
`1289`	`1291`
`1290`	`1292`	`An empty pipeline is considered fitted.`
`1291`	`1293`	`"""`
`1292`		`-`
`1293`	`1294`	`# First find the last step that is not 'passthrough'`
`1294`	`1295`	`last_step=None`
`1295`	`1296`	`for_,estimatorinreversed(self.steps):`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -282,6 +282,16 @@ def test_pipeline_invalid_parameters():`
`282`	`282`	`assertparams==params2`
`283`	`283`
`284`	`284`
	`285`	`+deftest_empty_pipeline():`
	`286`	`+X=iris.data`
	`287`	`+y=iris.target`
	`288`	`+`
	`289`	`+pipe=Pipeline([])`
	`290`	`+msg="The pipeline is empty. Please add steps."`
	`291`	`+withpytest.raises(ValueError,match=msg):`
	`292`	`+pipe.fit(X,y)`
	`293`	`+`
	`294`	`+`
`285`	`295`	`deftest_pipeline_init_tuple():`
`286`	`296`	`# Pipeline accepts steps as tuple`
`287`	`297`	`X=np.array([[1,2]])`

Comments

(0)