Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#25431 closed defect (fixed)

Timeouts in lattice precision

Reported by: Julian Rüth Owned by:
Priority: critical Milestone: sage-8.3
Component: padics Keywords:
Cc: Xavier Caruso, David Roe, Frédéric Chapoton Merged in:
Authors: Julian Rüth, Xavier Caruso Reviewers: Julian Rüth, David Loeffler
Report Upstream: N/A Work issues:
Branch: b80dc9e (Commits, GitHub, GitLab) Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Julian Rüth)

The patchbots time out with some lattice precision tests I have also seen the same issues with GitLab CI tests.

The test suite takes too much time, more precisely _test_matrix_smith.

sage: R = QpLC(7, label='init')
sage: %time TestSuite(R).run(skip='_test_teichmuller')
Wall time: 2min 59s
sage: %time R._test_matrix_smith()
Wall time: 2min 56s

Change History (15)

comment:1 Changed 5 years ago by Julian Rüth

Cc: Xavier Caruso David Roe Frédéric Chapoton added

comment:2 Changed 5 years ago by Julian Rüth

Authors: Julian Rüth
Description: modified (diff)

comment:3 Changed 5 years ago by Julian Rüth

Branch: u/saraedum/25431

comment:4 Changed 5 years ago by Julian Rüth

Commit: 16ecaab3129389cd5f2d7afb55ad949893c0f7ea
Status: newneeds_review

I have not run doctests yet, so let's see if the patchbots are happy now.

New commits:

16ecaabSkip _test_matrix_smith for lattice precision rings

comment:5 Changed 5 years ago by git

Commit: 16ecaab3129389cd5f2d7afb55ad949893c0f7eae67314c4adb29748b4ef683f351b1a1094c837de

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

e67314cSkip _test_matrix_smith for lattice precision rings

comment:6 Changed 5 years ago by Frédéric Chapoton

I have launched my patchbot having the timeout issue, to see if the branch brings any progress.

comment:7 Changed 5 years ago by Frédéric Chapoton

This indeed seems to fix the timeout issue (see petibonum report)

Note: my patchbot has found some pyflakes errors in the modified files, see the pyflakes plugin.

comment:8 Changed 5 years ago by Jeroen Demeyer

FYI: short doctests should take at most 1 second or so and long doctests at most 30 seconds (unless for exceptional reasons).

comment:9 Changed 5 years ago by Xavier Caruso

Branch: u/saraedum/25431u/caruso/25431

comment:10 Changed 5 years ago by Xavier Caruso

Commit: e67314c4adb29748b4ef683f351b1a1094c837deb80dc9e2d1996a76de2ef286d8860262cbfaaaa0

I fixed the pyflakes warnings.

New commits:

b80dc9eFix pyflakes errors

comment:11 Changed 5 years ago by Julian Rüth

Authors: Julian RüthJulian Rüth, Xavier Caruso

comment:12 Changed 5 years ago by Julian Rüth

Reviewers: Julian Rüth

Xavier, your changes look good. Can you review my original changes?

comment:13 Changed 5 years ago by David Loeffler

Reviewers: Julian RüthJulian Rüth, David Loeffler
Status: needs_reviewpositive_review

Changes look reasonable + it gets the timings down to an acceptable level. Positive review.

comment:14 Changed 5 years ago by Volker Braun

Branch: u/caruso/25431b80dc9e2d1996a76de2ef286d8860262cbfaaaa0
Resolution: fixed
Status: positive_reviewclosed

comment:15 Changed 5 years ago by Erik Bray

Commit: b80dc9e2d1996a76de2ef286d8860262cbfaaaa0

As noted on the mailing list, I am also getting

sage -t src/sage/rings/padics/
    Timed out (with segmentation fault after interrupt)

on cygwin. I will test this patch and see if the tests at least pass with these tests skipped by default (and I will also test with --long)

Note: See TracTickets for help on using tickets.