I'm pretty sure that removing it is the same as setting it to None, because it falls back to the superclass which sets unknown assumptions to None. I would set them both explicitly to None to be clear that that really is what we want, and not just that it isn't implemented.
Aaron Meurer On Sat, Mar 22, 2014 at 1:25 PM, Kalevi Suominen <[email protected]> wrote: > > > On Saturday, March 22, 2014 6:24:12 PM UTC+2, Aaron Meurer wrote: >> >> Oh, that's good. This was setting is_rational and is_irrational to >> None? I think that's what they should be. >> > I was testing with is_rational commented out (is_irrational does not exist > in Float). > A missing attribute would reveal every attempted use. I think this is how it > should > be in production code, to warn off any accidental use. > > Kalevi Suominen > > -- > You received this message because you are subscribed to the Google Groups > "sympy" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/sympy. > To view this discussion on the web visit > https://groups.google.com/d/msgid/sympy/ca368bca-abef-43ec-aba7-7ed55df20d76%40googlegroups.com. > > For more options, visit https://groups.google.com/d/optout. -- You received this message because you are subscribed to the Google Groups "sympy" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sympy. To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/CAKgW%3D6KwBEYoSKwRhbSL61o4N_pjNbgCij1FBPJcfAWpFxcaaA%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
