InstrumentalVariableRegression takes vs_prior_type and vs_hyperparams as fit/build_model arguments, so the InstrumentalVariable experiment stores both and forwards them through a # type: ignore[call-arg,union-attr] call that bypasses the model adapter. StateSpaceTimeSeries (#1112) takes the same two arguments in __init__: the prior type is validated at construction and InterruptedTimeSeries needs no forwarding, since experiments only call fit(X, y, coords) through the adapter.
Should the IV class move to constructor configuration too, so the experiment stops forwarding model kwargs around the adapter contract?
InstrumentalVariableRegressiontakesvs_prior_typeandvs_hyperparamsasfit/build_modelarguments, so theInstrumentalVariableexperiment stores both and forwards them through a# type: ignore[call-arg,union-attr]call that bypasses the model adapter.StateSpaceTimeSeries(#1112) takes the same two arguments in__init__: the prior type is validated at construction andInterruptedTimeSeriesneeds no forwarding, since experiments only callfit(X, y, coords)through the adapter.Should the IV class move to constructor configuration too, so the experiment stops forwarding model kwargs around the adapter contract?