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

add a finance chapter to the sage reference manual #9218

Closed
williamstein opened this issue Jun 11, 2010 · 11 comments
Closed

add a finance chapter to the sage reference manual #9218

williamstein opened this issue Jun 11, 2010 · 11 comments

Comments

@williamstein
Copy link
Contributor

This is a dependency for #9219.

Component: documentation

Author: William Stein, Minh Van Nguyen

Reviewer: Minh Van Nguyen, Karl-Dieter Crisman

Merged: sage-4.5.2.alpha0

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

@williamstein
Copy link
Contributor Author

comment:1

Attachment: trac_9218-finance_doc.patch.gz

The first patch adds the documentation. However the docstrings are all pre-rest, so look like crap. E.g., see

http://sage.math.washington.edu/home/wstein/build/sage-4.4.4.alpha0/devel/sage/doc/output/html/en/reference/finance.html

or just apply the patch, then do sage -docbuild reference html and look at output/html/en/reference/ for yourself.

A second patch will fix all the formatting.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jun 12, 2010

Attachment: trac_9218-reviewer.patch.gz

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jun 12, 2010

comment:2

Replying to @williamstein:

The first patch adds the documentation. However the docstrings are all pre-rest, so look like crap.

Not any more.

A second patch will fix all the formatting.

See my reviewer patch.

@sagetrac-mvngu

This comment has been minimized.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jun 12, 2010

Reviewer: Minh Van Nguyen

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jun 12, 2010

Author: William Stein

@kcrisman
Copy link
Member

Attachment: trac_9218-typo.patch.gz

Apply this last.

@kcrisman
Copy link
Member

Changed reviewer from Minh Van Nguyen to Minh Van Nguyen, Karl-Dieter Crisman

@kcrisman
Copy link
Member

comment:3

Looks good! Apply in the order finance_doc, reviewer, and typo. Also added Minh as an author, considering his patch is two orders of magnitude bigger than the original one.

@kcrisman
Copy link
Member

Changed author from William Stein to William Stein, Minh Van Nguyen

@qed777
Copy link
Mannequin

qed777 mannequin commented Jul 21, 2010

Merged: sage-4.5.2.alpha0

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

2 participants