Add some long time markers

saraedum commented 6 years ago

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]

Component: doctest framework

Author: Julian Rüth

Branch/Commit: u/saraedum/26119 @ ed4b9ea

Issue created by migration from https://trac.sagemath.org/ticket/26119

saraedum commented 6 years ago

Description changed:

--- 
+++ 
@@ -1 +1 @@
-
+I am adding some long time markers to tests that are not taking excessively long time but that are in files that are taking a long time overall.

saraedum commented 6 years ago

Branch: u/saraedum/26119

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Commit: 196cc51

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

`cb06b99`	`long time markers`
`196cc51`	`fixup`

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 196cc51 to db9a49e

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

db9a49e fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from db9a49e to 14bdbe1

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

14bdbe1 fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 14bdbe1 to 7c2bbfb

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

7c2bbfb fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 7c2bbfb to 11b18fe

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

11b18fe fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

0cbcefb fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 11b18fe to 0cbcefb

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

17048a5 fixup

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 0cbcefb to 17048a5

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

dc239ac long time markers

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 17048a5 to dc239ac

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

7a39c90 fix gdb doctest

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from dc239ac to 7a39c90

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

Changed commit from 7a39c90 to ed4b9ea

7ed8c4ca-6d56-4ae9-953a-41e42b4ed313 commented 6 years ago

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

ed4b9ea fix long time comment

saraedum commented 6 years ago

Description changed:

--- 
+++ 
@@ -1 +1,81 @@
-I am adding some long time markers to tests that are not taking excessively long time but that are in files that are taking a long time overall.
+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]
+```

jdemeyer commented 6 years ago

comment:14

Looks good, but how did you find those doctests?

saraedum commented 6 years ago

comment:15

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.

sagemath / sage

Add some long time markers #26119