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 argument unpacking #15

Open
dwf opened this issue May 27, 2014 · 0 comments
Open

support argument unpacking #15

dwf opened this issue May 27, 2014 · 0 comments

Comments

@dwf
Copy link
Contributor

dwf commented May 27, 2014

Like hyperopt.pyll's o_len.

It's a violation of namespace separation to pass it as a kwarg to partial/PartialPlus/etc. My preferred syntax would be one (possibly involving getitem on some magic object) that keeps output_length and any other metadata parameters in their own separate box from the partial's kwargs. I think this is doable with some care.

For the sake of not cluttering the graph too much, just add an attribute on a Node. This also lets as_partialplus more easily annotate known quantities.

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

No branches or pull requests

1 participant