Opened 3 years ago
Closed 3 years ago
#29440 closed enhancement (fixed)
Improve doctests of Degenerate Submanifold
Reported by:  ghkliem  Owned by:  

Priority:  major  Milestone:  sage9.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: 
Description
The doctests in sage t src/sage/manifolds/differentiable/degenerate_submanifold.py
have problems with time outs.
See for example:
 https://trac.sagemath.org/ticket/29104#comment:61
 https://groups.google.com/d/msg/sagerelease/eMQIlhglN98/bTwDGD9HBQAJ
 https://trac.sagemath.org/ticket/27122#comment:33
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
Branch:  → public/29440 

Commit:  → ef94ca59cc368c858958e408dcfa27c6ea05cd0c 
comment:2 Changed 3 years ago by
Status:  new → needs_review 

comment:3 Changed 3 years ago by
Commit:  ef94ca59cc368c858958e408dcfa27c6ea05cd0c → 848fec03b635a9f044b4f3daf39f45538a3f421e 

Branch pushed to git repo; I updated commit sha1. New commits:
848fec0  removed redundant import

comment:4 Changed 3 years ago by
Status:  needs_review → positive_review 

Replying to ghkliem:
The doctests in
sage t src/sage/manifolds/differentiable/degenerate_submanifold.py
have problems with time outs.See for example:
 https://trac.sagemath.org/ticket/29104#comment:61
 https://groups.google.com/d/msg/sagerelease/eMQIlhglN98/bTwDGD9HBQAJ
 https://trac.sagemath.org/ticket/27122#comment:33
Part of the problem might be, that the file makes no use of
long time
flags (testing takes 86 seconds on a fast computer withoutlong
).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:6 Changed 3 years ago by
Authors:  Jonathan Kliem 

Reviewers:  → Hans Fotsing Tetsing 
comment:7 Changed 3 years ago by
Authors:  → Jonathan Kliem 

comment:8 Changed 3 years ago by
Branch:  public/29440 → 848fec03b635a9f044b4f3daf39f45538a3f421e 

Resolution:  → fixed 
Status:  positive_review → closed 
New commits:
use # long time and add a smaller example