Opened 4 years ago

Closed 4 years ago

#25190 closed defect (fixed)

Some documentation improvements for LLL

Reported by: pbruin Owned by:
Priority: minor Milestone: sage-8.2
Component: documentation Keywords: LLL
Cc: Merged in:
Authors: Peter Bruin Reviewers: Simon Brandhorst
Report Upstream: N/A Work issues:
Branch: 55e9623 (Commits, GitHub, GitLab) Commit: 55e96238c22144789fa562dfcd82e03776192405
Dependencies: Stopgaps:

Status badges

Description

The documentation of sage.matrix.matrix_integer_dense.Matrix_integer_dense.LLL mentions an incorrect default value for the delta parameter. This ticket fixes this and makes some other improvements to the documentation of this method and the similar BKZ method.

Change History (3)

comment:1 Changed 4 years ago by pbruin

  • Branch set to u/pbruin/25190-LLL_documentation
  • Commit set to 55e96238c22144789fa562dfcd82e03776192405
  • Status changed from new to needs_review

comment:2 Changed 4 years ago by sbrandhorst

  • Reviewers set to Simon Brandhorst
  • Status changed from needs_review to positive_review

lgtm. doc builds.

comment:3 Changed 4 years ago by vbraun

  • Branch changed from u/pbruin/25190-LLL_documentation to 55e96238c22144789fa562dfcd82e03776192405
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.