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.

Sign up for GitHub

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

Jump to bottom

__radd__ doesn't work when left hand side is an Element #3774

Closed
robertwb opened this issue Aug 5, 2008 · 7 comments
Closed

__radd__ doesn't work when left hand side is an Element #3774

robertwb opened this issue Aug 5, 2008 · 7 comments
Assignees
@robertwb
Labels
c: coercion p: major / 3 t: bug
Milestone
sage-3.1.3

Comments

@robertwb
Copy link
Contributor

robertwb commented Aug 5, 2008 

On Aug 1, 2008, at 7:05 AM, Nils Skoruppa wrote:


It seems that,  for non elements,  __radd__ is set disfunctional
by the coercion model. On the other hand, it might be desirable
to have this enabled for people writing their own classes but having
reasons to avoid (parts of)  the coercion system (like me :-)

CC: @aghitza

Component: coercion

Issue created by migration from https://trac.sagemath.org/ticket/3774
@robertwb robertwb added this to the sage-3.1.3 milestone Aug 5, 2008
@robertwb robertwb self-assigned this Aug 5, 2008
@robertwb
Copy link
Contributor Author

robertwb commented Aug 5, 2008

comment:1

Attachment: 3774-radd.patch.gz

@robertwb
Copy link
Contributor Author

comment:2

Might depend on #3738.

@mwhansen
Copy link
Contributor

comment:3

This could use a doctest. robertwb, do you want to write one, if not, I can probably do it a bit later.

@robertwb
Copy link
Contributor Author

comment:4

Please go ahead and write one, though implementing __radd__ should not be encouraged.

@mwhansen
Copy link
Contributor

Attachment: trac_3774.patch.gz

@mwhansen
Copy link
Contributor

comment:6

Okay, the new patch should apply. Positive review.

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Sep 24, 2008

comment:7

Merged trac_3774.patch in Sage 3.1.3.alpha1

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Sep 24, 2008
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertwb robertwb

Labels
c: coercion p: major / 3 t: bug
Projects
None yet
Milestone
sage-3.1.3
Development

No branches or pull requests
2 participants
@mwhansen @robertwb