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.

Sign up for GitHub

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

Jump to bottom

Incorrect documentation for set difference #11366

Closed
kcrisman opened this issue May 21, 2011 · 9 comments
Closed

Incorrect documentation for set difference #11366

kcrisman opened this issue May 21, 2011 · 9 comments
Assignees
@sagetrac-mvngu
Labels
c: documentation p: major / 3 t: bug
Milestone
sage-5.6

Comments

@kcrisman
Copy link
Member

See this documentation, where the definition for set difference and set intersection is the same.

The actual doctests are correct, I think, it's just some sloppy copying or something in the definition. See enumerated sets for what seems to be correct.

Component: documentation

Author: Travis Scrimshaw

Reviewer: Julian Rueth

Merged: sage-5.6.beta1

Issue created by migration from https://trac.sagemath.org/ticket/11366
@kcrisman kcrisman added this to the sage-5.6 milestone May 21, 2011
@kcrisman

This comment has been minimized.

Sign in to view
@kcrisman kcrisman changed the title Incorrect documentation for enumerated set difference Incorrect documentation for set difference May 21, 2011
@tscrim
Copy link
Collaborator

tscrim commented Nov 17, 2012

Author: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Nov 17, 2012

comment:2

Made the quick change. Easy to review.

@saraedum
Copy link
Member

Reviewer: Julian Rueth

@saraedum
Copy link
Member

comment:3

A very minor problem, but I believe that the subject line of your patch should read: Trac #11366: Doc change for set difference in Set..

@tscrim
Copy link
Collaborator

tscrim commented Nov 22, 2012

Changes to commit message

@tscrim
Copy link
Collaborator

tscrim commented Nov 22, 2012

comment:4

Attachment: trac_11366-set_documentation-ts.patch.gz

Changed.

@tscrim
Copy link
Collaborator

tscrim commented Dec 16, 2012

comment:6

Thank you for the review.

@jdemeyer
Copy link

Merged: sage-5.6.beta1

@sagetrac-ljpk sagetrac-ljpk mannequin mentioned this issue Aug 16, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mvngu sagetrac-mvngu

Labels
c: documentation p: major / 3 t: bug
Projects
None yet
Milestone
sage-5.6
Development

No branches or pull requests
4 participants
@saraedum @tscrim @kcrisman @jdemeyer