#18264: make map_coefficients in modules_with_basis safer or more general
-------------------------------------+-------------------------------------
Reporter: mantepse | Owner:
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-6.7
Component: algebra | Resolution:
Keywords: map_coefficients, | Merged in:
modules_with_basis, base_ring, | Reviewers:
check arguments | Work issues:
Authors: Martin Rubey | Commit:
Report Upstream: N/A | 3a7761e95b8ca8b83e08635828a3754d751fe918
Branch: | Stopgaps:
u/mantepse/make_map_coefficients_in_modules_with_basis_safer_or_more_general|
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:10 mantepse]:
> 1. to be helpful for the casual user, it's default value would have to
be `True`
> - differently from what we have in `_from_dict`.
Yeah, I agree; it's not supper consistent. Yet `_from_dict` is a
private method when this one is public; so it's not unreasonable
either that the typical usage situation differ, inducing different
default behavior.
> 2. the docstring would then be something like
>
> {{{
> whether to try to coerce the result of applying `f` to the coefficient
ring.
> }}}
>
> which is a bit strange, given that we require `f` to be an
endofunction.
I see your point. We should be able to find some wording that makes it
sound natural. Maybe something like:
`Whether to systematically coerce to the coefficient ring the result
of applying `f`, for additional safety`.
From my perspective you can proceed! Feedback anyone else?
Cheers,
--
Ticket URL: <http://trac.sagemath.org/ticket/18264#comment:11>
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.
