Changes between Version 1 and Version 2 of Ticket #17130, comment 7


Ignore:
Timestamp:
10/10/14 10:29:09 (6 years ago)
Author:
jdemeyer
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #17130, comment 7

    v1 v2  
    1 I'm not surprised - the problem is that this kind of code was added at various times to handle certain cases where the output was not the same type as the input (e.g., complex output for real/`float` input). For instance, does the example mentioned in the code
     1I'm not surprised - the problem is that this kind of code was added at various times to handle certain cases where the output was not the same type as the input (e.g., complex output for real/float input). For instance, does the example mentioned in the code
    22{{{
    33arctan(RR(1))
    44}}}
    5 work properly with this patch?
     5work properly with this patch? Again, I'm sorry I don't have time to do more than read code for five minutes at this point :(
    66
    7 That example works, the problem lies with Python types:
    8 {{{
    9 sage: arctan(float(1))
    10 1/4*pi
    11 }}}
     7Edit: I see your most recent comment - yes, that would certainly be helpful, though I have a feeling a LOT of functions would have to be fixed... because some probably implicitly rely on this block but aren't thoroughly tested for unusual input/output.
     8
     9Edit by jdemeyer: Sorry, edit instead of reply