Opened 9 years ago
Closed 8 years ago
#16698 closed task (fixed)
remove deprecated prec arguments
Reported by: | rws | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | sage-duplicate/invalid/wontfix |
Component: | numerical | Keywords: | prec, keyword, deprecation, deprecated, beginner |
Cc: | Merged in: | ||
Authors: | André Apitzsch | Reviewers: | |
Report Upstream: | N/A | Work issues: | |
Branch: | u/aapitzsch/ticket/16698 (Commits, GitHub, GitLab) | Commit: | ba38202cf55caefb032e1b4519c96b9706577004 |
Dependencies: | Stopgaps: |
Description (last modified by )
There are still three instances of the prec keyword argument:
src/sage/functions/other.py: deprecation(7490, "The prec keyword argument is deprecated. Explicitly set the precision of the input, for example gamma(RealField(300)(1)), or use the prec argument to .n() for exact inputs, e.g., gamma(1).n(300), instead.") src/sage/functions/other.py: deprecation(7490, "Calling symbolic functions with arguments that cannot be coerced into symbolic expressions is deprecated.") src/sage/functions/log.py: deprecation(7490, "The prec keyword argument is deprecated. Explicitly set the precision of the input, for example exp(RealField(300)(1)), or use the prec argument to .n() for exact inputs, e.g., exp(1).n(300), instead.")
Change History (10)
comment:1 Changed 9 years ago by
Keywords: | beginner added |
---|---|
Priority: | major → trivial |
comment:2 Changed 9 years ago by
Milestone: | sage-6.3 → sage-6.4 |
---|
comment:3 Changed 8 years ago by
Authors: | → André Apitzsch |
---|---|
Branch: | → u/aapitzsch/ticket/16698 |
Commit: | → 2cf6f69523d1b750ea180407d7af402739f9459f |
comment:4 Changed 8 years ago by
Status: | new → needs_review |
---|
comment:5 Changed 8 years ago by
comment:6 Changed 8 years ago by
Description: | modified (diff) |
---|---|
Status: | needs_review → needs_info |
Please justify why you undo the deprecation of
sage: Q.<i> = NumberField(x^2+1) sage: gamma(i)
comment:7 Changed 8 years ago by
Commit: | 2cf6f69523d1b750ea180407d7af402739f9459f → ba38202cf55caefb032e1b4519c96b9706577004 |
---|
comment:8 Changed 8 years ago by
Description: | modified (diff) |
---|---|
Status: | needs_info → needs_review |
I removed TESTS::
. BTW the documentation of this function is not shown in the html reference documentation.
Replying to jdemeyer:
Please justify why you undo the deprecation of
sage: Q.<i> = NumberField(x^2+1) sage: gamma(i)
I misinterpreted the deprecation warning. Now an exception is raised.
comment:9 Changed 8 years ago by
Milestone: | sage-6.4 → sage-duplicate/invalid/wontfix |
---|---|
Status: | needs_review → positive_review |
Has been removed in #16023.
comment:10 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
Did you check that the documentation builds properly? This looks bad (double colon but block removed):
TESTS:: - sage: exp(2,prec=100) - doctest:...: DeprecationWarning: The prec keyword argument is deprecated. Explicitly set the precision of the input, for example exp(RealField(300)(1)), or use the prec argument to .n() for exact inputs, e.g., exp(1).n(300), instead. - See http://trac.sagemath.org/7490 for details. - 7.3890560989306502272304274606 -