Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for FunctionTransformer #458

Open
janvanrijn opened this issue Apr 27, 2018 · 2 comments
Open

Support for FunctionTransformer #458

janvanrijn opened this issue Apr 27, 2018 · 2 comments

Comments

@janvanrijn
Copy link
Member

I just came across the FunctionTransformer. Does the flow_to_sklearn / sklearn_to_flow support this module? If not, do we block uploading this?
(Just putting it out here, will write a MWE when I have time)

@amueller
Copy link
Contributor

amueller commented Oct 16, 2019

It seems to work. The function is pickled so it works as long as the function is available somewhere, just like any estimator.

I don't see a reason to block this. See https://openml.org/f/6554 for an example

@amueller
Copy link
Contributor

@mfeurer suggests adding an example and a unit test.
Using np.exp works but lambda x: np.exp(x) probably doesn't work without having the function defined and named somewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants