Opened 3 years ago

Closed 3 years ago

#29440 closed enhancement (fixed)

Improve doctests of Degenerate Submanifold

Reported by: gh-kliem Owned by:
Priority: major Milestone: sage-9.1
Component: geometry Keywords: degenerate manifolds, doctests
Cc: Hans Fotsing Tetsing, Eric Gourgoulhon, Matthias Köppe Merged in:
Authors: Jonathan Kliem Reviewers: Hans Fotsing Tetsing
Report Upstream: N/A Work issues:
Branch: 848fec0 (Commits, GitHub, GitLab) Commit: 848fec03b635a9f044b4f3daf39f45538a3f421e
Dependencies: Stopgaps:

Status badges

Description

The doctests in sage -t src/sage/manifolds/differentiable/degenerate_submanifold.py have problems with time outs.

See for example:

Part of the problem might be, that the file makes no use of long time flags (testing takes 86 seconds on a fast computer without --long).

We add those flags and some smaller examples to display functionality of the module. This brings down testing time without --long` to less than 30 seconds.

Change History (8)

comment:1 Changed 3 years ago by gh-kliem

Branch: public/29440
Commit: ef94ca59cc368c858958e408dcfa27c6ea05cd0c

New commits:

ef94ca5use # long time and add a smaller example

comment:2 Changed 3 years ago by gh-kliem

Status: newneeds_review

comment:3 Changed 3 years ago by git

Commit: ef94ca59cc368c858958e408dcfa27c6ea05cd0c848fec03b635a9f044b4f3daf39f45538a3f421e

Branch pushed to git repo; I updated commit sha1. New commits:

848fec0removed redundant import

comment:4 in reply to:  description Changed 3 years ago by Hans Fotsing Tetsing

Status: needs_reviewpositive_review

Replying to gh-kliem:

The doctests in sage -t src/sage/manifolds/differentiable/degenerate_submanifold.py have problems with time outs.

See for example:

Part of the problem might be, that the file makes no use of long time flags (testing takes 86 seconds on a fast computer without --long).

We add those flags and some smaller examples to display functionality of the module. This brings down testing time without --long` to less than 30 seconds.

Thank you so much for the improvement and the example!

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

please add missing reviewer full name

comment:6 Changed 3 years ago by Hans Fotsing Tetsing

Authors: Jonathan Kliem
Reviewers: Hans Fotsing Tetsing

comment:7 Changed 3 years ago by gh-kliem

Authors: Jonathan Kliem

comment:8 Changed 3 years ago by Volker Braun

Branch: public/29440848fec03b635a9f044b4f3daf39f45538a3f421e
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.