Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rewrite error functions and documentation #21819

Closed
rwst opened this issue Nov 4, 2016 · 34 comments
Closed

Rewrite error functions and documentation #21819

rwst opened this issue Nov 4, 2016 · 34 comments

Comments

@rwst
Copy link

rwst commented Nov 4, 2016

This ticket should accomplish:

http://dlmf.nist.gov/7

Component: symbolics

Author: Ralf Stephan

Branch/Commit: 1eac4d8

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/21819

@rwst rwst added this to the sage-7.5 milestone Nov 4, 2016
@rwst

This comment has been minimized.

@rwst
Copy link
Author

rwst commented Nov 9, 2016

Dependencies: #11349

@rwst rwst changed the title Symbolical erfc() implementation Rewrite error functions and documentation Nov 9, 2016
@rwst
Copy link
Author

rwst commented Nov 9, 2016

@rwst
Copy link
Author

rwst commented Nov 9, 2016

Changed dependencies from #11349 to none

@rwst
Copy link
Author

rwst commented Nov 9, 2016

New commits:

fbb77c021819: erf() reorg

@rwst

This comment has been minimized.

@rwst
Copy link
Author

rwst commented Nov 9, 2016

Author: Ralf Stephan

@rwst
Copy link
Author

rwst commented Nov 9, 2016

Commit: fbb77c0

@rwst

This comment has been minimized.

@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Nov 10, 2016

comment:5

The name error_fcn is nonstandard. I think it should be deprecated with an appropriate warning.

A couple minor things in the documentation in error.py:

  1. One LaTeX instance of \text should be \operatorname

  2. Five instances of code markdown in the AUTHORS section need double back ticks instead of single.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2016

Branch pushed to git repo; I updated commit sha1. New commits:

801c42aMerge branch 'develop' into t/21819/rewrite_error_functions_and_documentation
3c5739e21819: address reviewer's comments; maxima erfc bindings

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 10, 2016

Changed commit from fbb77c0 to 3c5739e

@fchapoton
Copy link
Contributor

comment:7

failing doctests

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 6, 2017

Changed commit from 3c5739e to be39e50

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 6, 2017

Branch pushed to git repo; I updated commit sha1. New commits:

2fb1d2dMerge branch 'develop' into t/21819/rewrite_error_functions_and_documentation
be39e5021819: fixes

@fchapoton
Copy link
Contributor

comment:10

does not apply

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 28, 2017

Changed commit from be39e50 to 7fade55

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 28, 2017

Branch pushed to git repo; I updated commit sha1. New commits:

7fade55Merge branch 'develop' into t/21819/rewrite_error_functions_and_documentation

@rwst
Copy link
Author

rwst commented Jan 28, 2017

New commits:

7fade55Merge branch 'develop' into t/21819/rewrite_error_functions_and_documentation

@rwst rwst modified the milestones: sage-7.5, sage-7.6 Jan 28, 2017
@rwst
Copy link
Author

rwst commented Apr 11, 2017

Changed branch from u/rws/rewrite_error_functions_and_documentation to u/rws/21819-1

@rwst
Copy link
Author

rwst commented Apr 11, 2017

Changed commit from 7fade55 to b56829c

@rwst
Copy link
Author

rwst commented Apr 11, 2017

comment:14

Squashed and lazy import added.


New commits:

b56829c21819: Rewrite error functions and documentation

@tscrim
Copy link
Collaborator

tscrim commented Apr 11, 2017

comment:15

In error.py, there is no reason to deprecate error_fcn there. I would do that in the original location. You should also have lazy_imports deprecations that redirect to the new location.

@rwst
Copy link
Author

rwst commented Apr 13, 2017

comment:16

Do you have an idea why the patchbot's startup_modules plugin still triggers?

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 13, 2017

Changed commit from b56829c to e96e7e1

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 13, 2017

Branch pushed to git repo; I updated commit sha1. New commits:

e96e7e121819: move error_fcn back to functions.other

@rwst
Copy link
Author

rwst commented Apr 13, 2017

comment:18

Replying to @tscrim:

... You should also have lazy_imports deprecations that redirect to the new location.

I tried but I don't think lazy_import's deprecation arg does what I think you're trying to do.

@rwst rwst modified the milestones: sage-7.6, sage-8.0 Apr 13, 2017
@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

Changed branch from u/rws/21819-1 to public/symbolics/rewrite_error_functions-21819

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

comment:20

I was going to try, but then I felt lazy and the current version works just fine. I made a few documentation tweaks. If my changes are good with you, then you can set a positive review.


New commits:

df32cc6Merge branch 'u/rws/21819-1' of git://trac.sagemath.org/sage into public/symbolics/rewrite_error_functions-21819
1eac4d8Some small tweaks to the doc.

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

Changed commit from e96e7e1 to 1eac4d8

@rwst
Copy link
Author

rwst commented May 16, 2017

comment:21

Looks good and tests in functions are fine. Thanks for the help with this.

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

comment:22

Sorry I let this drop off my radar.

@vbraun
Copy link
Member

vbraun commented May 18, 2017

Changed branch from public/symbolics/rewrite_error_functions-21819 to 1eac4d8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants