Ticket #3774 (closed defect: fixed)

Opened 4 months ago

Last modified 2 months ago

[with patch, positive review] __radd__ doesn't work when left hand side is an Element

Reported by: robertwb Assigned to: robertwb
Priority: major Milestone: sage-3.1.3
Component: coercion Keywords:
Cc: AlexGhitza

Description 

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 :-)

Attachments

3774-radd.patch (1.2 kB) - added by robertwb on 08/05/2008 01:25:22 AM.
trac_3774.patch (2.2 kB) - added by mhansen on 09/23/2008 07:10:59 PM.

Change History

08/05/2008 01:25:22 AM changed by robertwb

  • attachment 3774-radd.patch added.

08/05/2008 01:25:54 AM changed by robertwb

  • summary changed from __radd__ doesn't work when left hand side is an Element to [with patch, needs review] __radd__ doesn't work when left hand side is an Element.

08/11/2008 09:33:01 AM changed by robertwb

Might depend on #3738.

08/24/2008 05:02:30 PM changed by mhansen

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

08/25/2008 09:58:41 AM changed by robertwb

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

09/20/2008 04:34:35 PM changed by AlexGhitza

  • cc set to AlexGhitza.

09/23/2008 07:10:59 PM changed by mhansen

  • attachment trac_3774.patch added.

09/23/2008 07:11:26 PM changed by mhansen

  • summary changed from [with patch, needs review] __radd__ doesn't work when left hand side is an Element to [with patch, positive review] __radd__ doesn't work when left hand side is an Element.

Okay, the new patch should apply. Positive review.

09/23/2008 09:23:27 PM changed by mabshoff

  • status changed from new to closed.
  • resolution set to fixed.

Merged trac_3774.patch in Sage 3.1.3.alpha1