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

Improve @parallel documentation to give examples of functions with multiple arguments #11462

Closed
nilesjohnson mannequin opened this issue Jun 10, 2011 · 10 comments
Closed

Comments

@nilesjohnson
Copy link
Mannequin

nilesjohnson mannequin commented Jun 10, 2011

All of the examples for @parallel (even those mentioned on mailing lists) demonstrate using it with functions that take a single argument. Figuring out the syntax for multiple-argument functions can be slightly non-obvious, so the documentation should be improved.

More details at this asksage question.

CC: @kcrisman

Component: documentation

Keywords: @parallel documentation beginner sd35.5

Author: Nathan Carter

Reviewer: Karl-Dieter Crisman

Merged: sage-5.0.beta1

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

@nilesjohnson nilesjohnson mannequin added this to the sage-4.8 milestone Jun 10, 2011
@nilesjohnson nilesjohnson mannequin added the t: bug label Jun 10, 2011
@nathancarter
Copy link
Mannequin

nathancarter mannequin commented Jan 9, 2012

comment:2

Will post in a moment a patch that does what this ticket asks...

@nathancarter nathancarter mannequin added the s: needs review label Jan 9, 2012
@nathancarter
Copy link
Mannequin

nathancarter mannequin commented Jan 9, 2012

Changed keywords from @parallel documentation beginner to @parallel documentation beginner sd35.5

@nathancarter
Copy link
Mannequin

nathancarter mannequin commented Jan 9, 2012

Attachment: trac-11462-parallel-documentation.patch.gz

@nathancarter
Copy link
Mannequin

nathancarter mannequin commented Jan 9, 2012

comment:4

Updated patch to ensure doctests succeed.

@kcrisman
Copy link
Member

comment:5

Good work, looks good in the manual, all is correct and passes tests. Positive review.

@kcrisman
Copy link
Member

Author: Nathan Carter

@kcrisman
Copy link
Member

Reviewer: Karl-Dieter Crisman

@jdemeyer
Copy link

Changed work issues from improve documentation to none

@jdemeyer jdemeyer modified the milestones: sage-4.8, sage-5.0 Jan 10, 2012
@kcrisman
Copy link
Member

comment:7

See also #11461.

@jdemeyer
Copy link

Merged: sage-5.0.beta1

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