Closed lftabera closed 10 years ago
It is a one-line fix.
By the way, if a binary method with keyword values that is not decorated
__op__(self, y, method='z')
One can call it with x.__op__(y, 'preferred_method')
But once it is decorated this is not possible and key=value arguments are mandatory. Is this inteded?
Author: Luis Felipe Tabera Alonso
Reviewer: Robert Bradshaw
Looks good to me.
Lets make sure the tests pass though.
Robert,
Could you please take a look to the ticket? It is a one liner, you claimed that looked good and after seven months, patchbot has not complained.
Branch: u/lftabera/ticket/14186
Commit: 0cb642b
Branch pushed to git repo; I updated commit sha1. New commits:
0cb642b | Trac #14186 coerce_binop errors with keyword arguments |
Looks good to me.
Do not forget to add yoursef to the reviewers field
Changed reviewer from Robert Bradshaw to Robert Bradshaw, Travis Scrimshaw
On a coerced binary operator
Component: coercion
Keywords: coerce_binop, keyword
Author: Luis Felipe Tabera Alonso
Branch/Commit: u/lftabera/ticket/14186 @
0cb642b
Reviewer: Robert Bradshaw, Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/14186