confounds icon indicating copy to clipboard operation
confounds copied to clipboard

better validation of inputs to Deconfounders

Open raamana opened this issue 2 years ago • 13 comments

the #19 reminds me of how some users can be confused given the code lets the second argument to .fit() and .transform() optional with y=None. The only reason we have y=None is to try follow sklearn conventions and to pass their tests, but given we can't pass them anyway, we should tighten them up and make it an error to not supply the second [necessary] input argument.

cc @jrasero @jameschapman19

raamana avatar Apr 29 '22 13:04 raamana