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

Fix reference in eliminate_linear_variables #10283

Closed
malb opened this issue Nov 17, 2010 · 6 comments
Closed

Fix reference in eliminate_linear_variables #10283

malb opened this issue Nov 17, 2010 · 6 comments

Comments

@malb
Copy link
Member

malb commented Nov 17, 2010

Stanislav Bulyin reported in private communication that the reference given in the documentation for eliminate_linear_variables() is wrong.

Apply:

  1. attachment: trac_10283.patch
  2. attachment: trac-10283_reviewer.patch

Component: cryptography

Author: Martin Albrecht

Reviewer: Minh Van Nguyen

Merged: sage-4.6.1.alpha2

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

@malb
Copy link
Member Author

malb commented Nov 17, 2010

Attachment: trac_10283.patch.gz

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Nov 18, 2010

Reviewer: Minh Van Nguyen

@sagetrac-mvngu

This comment has been minimized.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Nov 18, 2010

comment:2

Attachment: trac-10283_reviewer.patch.gz

Applying attachment: trac_10283.patch to Sage 4.6.1.alpha1, building the HTML version of the reference manual results in the following warning:

/dev/shm/mvngu/sage-4.6.1.alpha1/devel/sage/doc/en/reference/sage/crypto/mq/mpolynomialsystem.rst:: WARNING: citation not found: CB07

And building the PDF version results in the following warning:

resolving references...
/dev/shm/mvngu/sage-4.6.1.alpha1/devel/sage/doc/en/reference/index.rst:: WARNING: citation not found: CB07

The warning is fixed in attachment: trac-10283_reviewer.patch, which needs reviewing.

@malb
Copy link
Member Author

malb commented Nov 18, 2010

comment:3

Looks good, thanks for spotting that!

@jdemeyer
Copy link

Merged: sage-4.6.1.alpha2

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