Opened 9 years ago

Closed 9 years ago

#10283 closed defect (fixed)

Fix reference in eliminate_linear_variables

Reported by: malb Owned by: mvngu
Priority: trivial Milestone: sage-4.6.1
Component: cryptography Keywords:
Cc: Merged in: sage-4.6.1.alpha2
Authors: Martin Albrecht Reviewers: Minh Van Nguyen
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by mvngu)

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

Apply:

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

Attachments (2)

trac_10283.patch (1.2 KB) - added by malb 9 years ago.
trac-10283_reviewer.patch (620 bytes) - added by mvngu 9 years ago.

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by malb

comment:1 Changed 9 years ago by malb

  • Status changed from new to needs_review

Changed 9 years ago by mvngu

comment:2 Changed 9 years ago by mvngu

  • Description modified (diff)
  • Reviewers set to Minh Van Nguyen

Applying 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 trac-10283_reviewer.patch, which needs reviewing.

comment:3 Changed 9 years ago by malb

  • Status changed from needs_review to positive_review

Looks good, thanks for spotting that!

comment:4 Changed 9 years ago by jdemeyer

  • Merged in set to sage-4.6.1.alpha2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.