#20162: properties of converted finite field elements are wrong
-------------------------------------+-------------------------------------
Reporter: behackl | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-7.1
Component: symbolics | Resolution:
Keywords: | Merged in:
Authors: Benjamin Hackl | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/behackl/symbolic/finite-field- | 591899ba8eda202cf8af3a2c22ef7aab85059ccf
reals | Stopgaps:
Dependencies: pynac-0.6.5 |
-------------------------------------+-------------------------------------
Comment (by behackl):
Replying to [comment:9 mmezzarobba]:
> I doubt it is such a good idea to special-case finite fields here. Why
does `py_imag` default to returning `True` in the first place?!
This is because if in doubt `py_is_real` asks `py_imag` whether the
imaginary part is 0---which is the case for these elements, because if in
doubt, `py_imag` assumes that the imaginary part is 0. ;-)
> And as for returning results based on the parent of the wrapped object,
I'd be tempted to base the logic on whether the parent has coercions to
things like `{Real,Complex}Field(mpfr_prec_min())` instead of having
explicit lists of “real” and “non-real” parents.
This might be an approach worth pursuing.
--
Ticket URL: <http://trac.sagemath.org/ticket/20162#comment:10>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" 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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
