-
Notifications
You must be signed in to change notification settings - Fork 367
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
by
with arrays of inputs and broadcasting
#1615
Comments
I guess this could be handled by merging the vectors/tuples/pairs. |
I thought that this was intentional, but we can add merging as @nalimilan writes. The rules would be to merge:
|
I agree that named-tuple for broadcasted arguments is not necessary. But I think that this should be allowed. Yes I think merging should do the trick, we can discuss more in depth once I make a PR/ |
@pdeffebach When you write a PR please take this behavior into account:
so duplicates are silently overwritten. We do not want this I think and an error should be thrown in case of a duplicate. |
This is done, so I am closing this. We are missing |
It looks like the splatting doesn't quite work with having vectors of columns you want to act on.
I'm done with grad apps now so I will see if I can poke around for a fix soon.
The text was updated successfully, but these errors were encountered: