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

Add Support for Providing IO Size for Primitives #54

Open
proska opened this issue Jul 10, 2018 · 0 comments
Open

Add Support for Providing IO Size for Primitives #54

proska opened this issue Jul 10, 2018 · 0 comments

Comments

@proska
Copy link
Contributor

proska commented Jul 10, 2018

Some primitives like "d3m.primitives.common_primitives.FeedForwardNeuralNet" require the Input and output size to be specified as hyper-parameters. Currently, our system does not infer the data size during pipeline generation in "to_pipeline" method which makes adding support for this particular set of primitives a little challenging.
At this version of the code, we change configuration points to pipeline solely based on pipeline topological information. It works well with sklearn primitives as they internally infer the IO size during the fit. However, in other primitives, the behavior is not consistent.
We have to either completely ignore these primitives or evaluate the size of primitive outputs during pipeline execution.

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

2 participants