#19152: {Real,Complex}Ball: Miscellaneous fixes and improvements
-------------------------------------+-------------------------------------
Reporter: mmezzarobba | Owner:
Type: enhancement | Status: needs_info
Priority: major | Milestone: sage-6.10
Component: numerical | Resolution:
Keywords: | Merged in:
Authors: Marc Mezzarobba | Reviewers: Clemens Heuberger
Report Upstream: N/A | Work issues:
Branch: public/19152-arb- | Commit:
misc | ea08942380a0cfab4614de252bac9a8db340b3b4
Dependencies: #19063 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jdemeyer):
Replying to [comment:17 mmezzarobba]:
> Replying to [comment:12 cheuberg]:
> > 1. `arf.pxd`: You define `arf_rnd_t` as an enum and rely on the fact
that the numeric values of `ARF_RND_DOWN` etc. will never change in the C
header files. However, there, the values are macros, so I do not know
whether we can safely assume the values never to change.
>
> I fear I don't understand what you mean here, sorry. I thought that
cython would insert textual references to `ARF_RND_*` in the generated C
code. Isn't that the case?
I agree, there is no issue here. Cython doesn't care about the values.
> > 5. `RealBall.upper`, `RealBall.lower`, `RealBall.endpoints`: missing
INPUT: and OUTPUT: sections, document and test parameter `rnd`
> The main options I see are:
> - either to return the endpoints rounded (in which direction? we can't
be 100% compatible with interval fields!) to the parent's precision,
I would go for: rounding endpoints in the obvious direction (i.e. up for
the upper endpoint and down for the lower endpoint)
--
Ticket URL: <http://trac.sagemath.org/ticket/19152#comment:21>
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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
