Opened 4 years ago

Last modified 4 years ago

#26119 new task

Add some long time markers

Reported by: saraedum Owned by:
Priority: major Milestone: sage-8.4
Component: doctest framework Keywords:
Cc: Merged in:
Authors: Julian Rüth Reviewers:
Report Upstream: N/A Work issues:
Branch: u/saraedum/26119 (Commits, GitHub, GitLab) Commit: ed4b9ea3bc3f96204b308e807531568307bce54a
Dependencies: Stopgaps:

Status badges

Description (last modified by saraedum)

In particular this adds markers to tests that are not taking excessively long but that are in files that are taking a long time overall.

Feel free to push to this ticket if you don't like any of the markers.

Before/After?

algebras/steenrod/steenrod_algebra.py [-[683-] {+[681+} tests, [-15.53-] {+5.68+} s]
categories/category.py [-[437-] {+[436+} tests, [-5.89-] {+1.91+} s]
categories/finite_monoids.py [-[52-] {+[51+} tests, [-19.75-] {+9.37+} s]
categories/pushout.py [-[871-] {+[870+} tests, [-13.28-] {+2.82+} s]
coding/linear_code.py [-[697-] {+[694+} tests, [-26.12-] {+9.27+} s]
combinat/crystals/affine_factorization.py [-[86-] {+[85+} tests, [-22.85-] {+13.05+} s]
combinat/designs/database.py [-[358-] {+[355+} tests, [-22.70-] {+11.49+} s]
combinat/designs/orthogonal_arrays_find_recursive.pyx [-[60-] {+[58+} tests, [-5.83-] {+0.77+} s]
combinat/root_system/weight_space.py [-[92-] {+[91+} tests, [-5.64-] {+2.41+} s]
combinat/skew_tableau.py [-[361-] {+[360+} tests, [-11.34-] {+4.29+} s]
combinat/species/generating_series.py [-[217-] {+[216+} tests, [-7.77-] {+1.14+} s]
crypto/sboxes.py [-[2 tests, 8.85-] {+[1 test, 0.41+} s]
crypto/sbox.py [-[237-] {+[234 tests, 4.08 s]
doctest/forker.py [447+} tests, [-17.49-] {+15.76+} s]
functions/exp_integral.py [-[261-] {+[260+} tests, [-12.56-] {+8.42+} s]
geometry/polyhedron/base.py [-[1061-] {+[1059+} tests, [-46.11-] {+30.96+} s]
graphs/generic_graph.py [-[3135-] {+[3133+} tests, [-24.17-] {+21.51+} s]
groups/finitely_presented_named.py [-[78-] {+[77+} tests, [-13.56-] {+5.94+} s]
groups/finitely_presented.py [-[327-] {+[326+} tests, [-11.00-] {+7.99+} s]
groups/libgap_mixin.py [-[112-] {+[111+} tests, [-19.36-] {+3.77+} s]
groups/perm_gps/permgroup.py [-[837-] {+[836+} tests, [-17.40-] {+13.54+} s]
homology/homology_vector_space_with_basis.py [-[193-] {+[188+} tests, [-12.98-] {+6.15+} s]
interfaces/maxima_abstract.py [-[233-] {+[232+} tests, [-28.91-] {+7.17+} s]
manifolds/differentiable/affine_connection.py [-[426-] {+[401+} tests, [-77.00-] {+42.10+} s]
manifolds/differentiable/diff_form.py [-[368-] {+[363+} tests, [-37.45-] {+35.80+} s]
manifolds/differentiable/integrated_curve.py [-[380-] {+[379+} tests, [-46.22-] {+41.53+} s]
manifolds/differentiable/levi_civita_connection.py [-[108-] {+[100+} tests, [-34.19-] {+15.80+} s]
manifolds/differentiable/manifold_homset.py [-[344-] {+[338+} tests, [-21.80-] {+16.18+} s]
manifolds/differentiable/metric.py [-[450-] {+[408+} tests, [-77.80-] {+54.05+} s]
manifolds/differentiable/multivectorfield.py [-[297-] {+[287+} tests, [-57.65-] {+35.60+} s]
manifolds/differentiable/pseudo_riemannian.py [-[140-] {+[135+} tests, [-20.80-] {+15.23+} s]
manifolds/differentiable/pseudo_riemannian_submanifold.py [330 tests, [-57.03-] {+58.26+} s]
manifolds/differentiable/scalarfield.py [-[263-] {+[260+} tests, [-22.37-] {+15.13+} s]
manifolds/differentiable/tensorfield.py [-[889-] {+[886+} tests, [-110.73-] {+99.65+} s]
manifolds/differentiable/vectorfield.py [-[241-] {+[236+} tests, [-34.01-] {+21.78+} s]
matrix/matrix_gfpn_dense.pyx [-[189-] {+[186+} tests, [-41.45-] {+2.39+} s]
matrix/matrix_integer_dense_hnf.py [-[125-] {+[124+} tests, [-43.14-] {+1.20+} s]
matrix/matrix_space.py [-[381-] {+[379+} tests, [-8.21-] {+5.85+} s]
misc/functional.py [-[345-] {+[343+} tests, [-18.17-] {+6.85+} s]
modular/arithgroup/congroup_gammaH.py [-[158-] {+[157+} tests, [-6.37-] {+2.66+} s]
modular/local_comp/type_space.py [-[91-] {+[88+} tests, [-22.28-] {+10.78+} s]
modular/modform/ambient_R.py [-[26-] {+[25+} tests, [-16.70-] {+0.76+} s]
modular/modform/cuspidal_submodule.py [-[60-] {+[59+} tests, [-9.62-] {+2.59+} s]
modular/modform/element.py [-[435-] {+[427+} tests, [-34.92-] {+18.84+} s]
modular/modform_hecketriangle/hecke_triangle_group_element.py [-[735-] {+[734+} tests, [-31.60-] {+28.97+} s]
parallel/decorate.py [-[90-] {+[89+} tests, [-28.74-] {+5.36+} s]
plot/graphics.py [-[432-] {+[430+} tests, [-49.99-] {+40.18+} s]
plot/plot3d/implicit_plot3d.py [-[91-] {+[90+} tests, [-26.19-] {+22.22+} s]
quadratic_forms/ternary_qf.py [-[323-] {+[320+} tests, [-10.50-] {+10.05+} s]
rings/asymptotic/asymptotic_expansion_generators.py [-[136-] {+[135+} tests, [-20.50-] {+18.16+} s]
rings/finite_rings/finite_field_constructor.py [-[116-] {+[115+} tests, [-10.89-] {+1.03+} s]
rings/number_field/number_field_element.pyx [-[1099-] {+[1098+} tests, [-16.59-] {+12.10+} s]
rings/number_field/number_field.py [-[2028-] {+[2027+} tests, [-30.81-] {+21.09+} s]
rings/padics/padic_base_leaves.py [-[212-] {+[210+} tests, [-35.83-] {+7.50+} s]
rings/padics/padic_extension_leaves.py [-[72-] {+[67+} tests, [-50.14-] {+7.68+} s]
rings/polynomial/multi_polynomial_sequence.py [-[240-] {+[239+} tests, [-30.09-] {+9.86+} s]
rings/tests.py [-[36-] {+[35+} tests, [-21.38-] {+3.26+} s]
rings/valuation/valuation.py [-[238-] {+[237+} tests, [-28.67-] {+5.27+} s]
schemes/elliptic_curves/ell_curve_isogeny.py [-[987-] {+[985+} tests, [-25.81-] {+10.68+} s]
schemes/elliptic_curves/ell_number_field.py [-[761-] {+[734+} tests, [-95.41-] {+41.29+} s]
schemes/elliptic_curves/ell_rational_field.py [-[834-] {+[830+} tests, [-44.92-] {+30.30+} s]
schemes/elliptic_curves/gal_reps_number_field.py [-[188-] {+[179+} tests, [-50.30-] {+31.81+} s]
schemes/elliptic_curves/heegner.py [-[1095-] {+[1092+} tests, [-31.22-] {+24.00+} s]
schemes/elliptic_curves/height.py [-[319-] {+[318+} tests, [-24.75-] {+10.22+} s]
schemes/elliptic_curves/isogeny_class.py [-[152-] {+[132+} tests, [-55.56-] {+16.18+} s]
schemes/elliptic_curves/isogeny_small_degree.py [-[275-] {+[272+} tests, [-26.90-] {+15.58+} s]
schemes/elliptic_curves/saturation.py [-[23-] {+[21+} tests, [-9.89-] {+3.48+} s]
schemes/hyperelliptic_curves/hyperelliptic_finite_field.py [-[365-] {+[364+} tests, [-16.73-] {+8.61+} s]
schemes/product_projective/subscheme.py [-[67-] {+[66+} tests, [-8.40-] {+1.59+} s]
schemes/riemann_surfaces/riemann_surface.py [-[306-] {+[297+} tests, [-22.49-] {+14.98+} s]
structure/coerce_dict.pyx [-[312-] {+[311+} tests, [-25.90-] {+3.37+} s]
tests/cmdline.py [-[253-] {+[252+} tests, [-77.86-] {+1 failure, 48.61+} s]
tests/french_book/mpoly.py [-[147-] {+[146+} tests, [-17.36-] {+6.09+} s]

Change History (15)

comment:1 Changed 4 years ago by saraedum

  • Description modified (diff)

comment:2 Changed 4 years ago by saraedum

  • Branch set to u/saraedum/26119

comment:3 Changed 4 years ago by git

  • Commit set to 196cc511bbb628c78d2883c88358353239f71739

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

cb06b99long time markers
196cc51fixup

comment:4 Changed 4 years ago by git

  • Commit changed from 196cc511bbb628c78d2883c88358353239f71739 to db9a49eec5fa645b0a0964cb9221dcc21428c0d4

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

db9a49efixup

comment:5 Changed 4 years ago by git

  • Commit changed from db9a49eec5fa645b0a0964cb9221dcc21428c0d4 to 14bdbe195c4b833548f0c772c8c85aafb95d76b4

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

14bdbe1fixup

comment:6 Changed 4 years ago by git

  • Commit changed from 14bdbe195c4b833548f0c772c8c85aafb95d76b4 to 7c2bbfbf74e9cc3ac511ae70f40cf2f2f78aa72c

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

7c2bbfbfixup

comment:7 Changed 4 years ago by git

  • Commit changed from 7c2bbfbf74e9cc3ac511ae70f40cf2f2f78aa72c to 11b18fef034f757ed9253aeafd513d787c568fee

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

11b18fefixup

comment:8 Changed 4 years ago by git

  • Commit changed from 11b18fef034f757ed9253aeafd513d787c568fee to 0cbcefbd25850b40d8ce7f2cf0bb8ae55172508a

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

0cbcefbfixup

comment:9 Changed 4 years ago by git

  • Commit changed from 0cbcefbd25850b40d8ce7f2cf0bb8ae55172508a to 17048a562967b87521e3c60ca5d5839baf26dcb9

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

17048a5fixup

comment:10 Changed 4 years ago by git

  • Commit changed from 17048a562967b87521e3c60ca5d5839baf26dcb9 to dc239ac5c3ffc4f150a21388ce1c11503e6b6e3a

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

dc239aclong time markers

comment:11 Changed 4 years ago by git

  • Commit changed from dc239ac5c3ffc4f150a21388ce1c11503e6b6e3a to 7a39c90a4ef77df3caee5f6d457c0f1f82b62b04

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

7a39c90fix gdb doctest

comment:12 Changed 4 years ago by git

  • Commit changed from 7a39c90a4ef77df3caee5f6d457c0f1f82b62b04 to ed4b9ea3bc3f96204b308e807531568307bce54a

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

ed4b9eafix long time comment

comment:13 Changed 4 years ago by saraedum

  • Description modified (diff)

comment:14 Changed 4 years ago by jdemeyer

Looks good, but how did you find those doctests?

comment:15 Changed 4 years ago by saraedum

Some creative grepping. I got my hands on a 160 core machine so I could run doctests quickly many times…see #26118. Note that some of these turned out not to actually make a difference. Either because the computation that took a long time is cached and now it shows up somewhere else or because the runtime apparently only is bad when you run lots of doctests in parallel.

I'll revert some of the changes accordingly.

Note: See TracTickets for help on using tickets.