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

follow-up to #6749: fix warnings when building reference manual #6916

Closed
sagetrac-mvngu mannequin opened this issue Sep 10, 2009 · 6 comments
Closed

follow-up to #6749: fix warnings when building reference manual #6916

sagetrac-mvngu mannequin opened this issue Sep 10, 2009 · 6 comments

Comments

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Sep 10, 2009

As the subject says. This is a follow-up to ticket #6749.

CC: @nathanncohen

Component: documentation

Author: Minh Van Nguyen

Reviewer: William Stein, John Palmieri

Merged: Sage 4.1.2.rc0

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

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Sep 10, 2009

Author: Minh Van Nguyen

@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.1.2 milestone Sep 10, 2009
@jhpalmieri
Copy link
Member

Reviewer: William Stein, John Palmieri

@jhpalmieri
Copy link
Member

comment:3

I give it a positive review, too, but if you want to change the lines

    - ``seq`` -- Two different possible types: 
      - A sequence of pairs (weight, value). 
      - A sequence of reals (a value of 1 is assumed). 

by adding a blank line after the first one, it looks a little better (to me) when typeset. I'll give that a positive review in advance, so if you want to make the change, no need for more review.

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Sep 30, 2009

Attachment: trac_6916-typos.patch.gz

based on Sage 4.1.2.alpha4

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Sep 30, 2009

comment:4

Replying to @jhpalmieri:

I give it a positive review, too, but if you want to change the lines

    - ``seq`` -- Two different possible types: 
      - A sequence of pairs (weight, value). 
      - A sequence of reals (a value of 1 is assumed). 

by adding a blank line after the first one, it looks a little better (to me) when typeset.

Done.

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented Sep 30, 2009

Merged: Sage 4.1.2.rc0

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