Opened 3 years ago
Last modified 3 years ago
#26119 new task
Add some long time markers
Reported by:  saraedum  Owned by:  

Priority:  major  Milestone:  sage8.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: 
Description (last modified by )
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.
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 3 years ago by
 Description modified (diff)
comment:2 Changed 3 years ago by
 Branch set to u/saraedum/26119
comment:3 Changed 3 years ago by
 Commit set to 196cc511bbb628c78d2883c88358353239f71739
comment:4 Changed 3 years ago by
 Commit changed from 196cc511bbb628c78d2883c88358353239f71739 to db9a49eec5fa645b0a0964cb9221dcc21428c0d4
Branch pushed to git repo; I updated commit sha1. New commits:
db9a49e  fixup

comment:5 Changed 3 years ago by
 Commit changed from db9a49eec5fa645b0a0964cb9221dcc21428c0d4 to 14bdbe195c4b833548f0c772c8c85aafb95d76b4
Branch pushed to git repo; I updated commit sha1. New commits:
14bdbe1  fixup

comment:6 Changed 3 years ago by
 Commit changed from 14bdbe195c4b833548f0c772c8c85aafb95d76b4 to 7c2bbfbf74e9cc3ac511ae70f40cf2f2f78aa72c
Branch pushed to git repo; I updated commit sha1. New commits:
7c2bbfb  fixup

comment:7 Changed 3 years ago by
 Commit changed from 7c2bbfbf74e9cc3ac511ae70f40cf2f2f78aa72c to 11b18fef034f757ed9253aeafd513d787c568fee
Branch pushed to git repo; I updated commit sha1. New commits:
11b18fe  fixup

comment:8 Changed 3 years ago by
 Commit changed from 11b18fef034f757ed9253aeafd513d787c568fee to 0cbcefbd25850b40d8ce7f2cf0bb8ae55172508a
Branch pushed to git repo; I updated commit sha1. New commits:
0cbcefb  fixup

comment:9 Changed 3 years ago by
 Commit changed from 0cbcefbd25850b40d8ce7f2cf0bb8ae55172508a to 17048a562967b87521e3c60ca5d5839baf26dcb9
Branch pushed to git repo; I updated commit sha1. New commits:
17048a5  fixup

comment:10 Changed 3 years ago by
 Commit changed from 17048a562967b87521e3c60ca5d5839baf26dcb9 to dc239ac5c3ffc4f150a21388ce1c11503e6b6e3a
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
dc239ac  long time markers

comment:11 Changed 3 years ago by
 Commit changed from dc239ac5c3ffc4f150a21388ce1c11503e6b6e3a to 7a39c90a4ef77df3caee5f6d457c0f1f82b62b04
Branch pushed to git repo; I updated commit sha1. New commits:
7a39c90  fix gdb doctest

comment:12 Changed 3 years ago by
 Commit changed from 7a39c90a4ef77df3caee5f6d457c0f1f82b62b04 to ed4b9ea3bc3f96204b308e807531568307bce54a
Branch pushed to git repo; I updated commit sha1. New commits:
ed4b9ea  fix long time comment

comment:13 Changed 3 years ago by
 Description modified (diff)
comment:14 Changed 3 years ago by
Looks good, but how did you find those doctests?
comment:15 Changed 3 years ago by
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.
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
long time markers
fixup