sagemath/sage

Fix incorrect doctest introduced in #29978

kliem opened this issue · 7 comments

kliem commented

From #32544:

sage -t --long --random-seed=23747002002644886606785003174022326205 src/sage/modules/free_module_integer.py
**********************************************************************
File "src/sage/modules/free_module_integer.py", line 370, in sage.modules.free_module_integer.FreeModule_submodule_with_basis_integer.LLL
Failed example:
    new_min <= old_min
Expected:
    True
Got:
    False

CC: @mwageringel

Component: modular forms

Author: Jonathan Kliem

Branch/Commit: 0e87ca6

Reviewer: Markus Wageringel

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

kliem commented

Commit: 0e87ca6

kliem commented

Branch: public/32771

kliem commented

New commits:

0e87ca6fix incorrect doctest from #29978
mwageringel commented

Reviewer: Markus Wageringel

mwageringel commented
comment:3

LGTM.

kliem commented
comment:4

Thank you.

vbraun commented

Changed branch from public/32771 to 0e87ca6