wrong output of %timeit

[zimmermann6]

sage: R=RealField(1000000)
sage: a=R(1/sqrt(3))
sage: %timeit a.arcsin()
1 loops, best of 3: 7.26 s per loop

It should be 1 loop, best of 3.

Upstream: Reported upstream. No feedback yet.

Component: user interface

Author: Saket Bhatt

Branch/Commit: bc52a1c

Reviewer: Paul Zimmermann

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

[Bhatt21]

comment:5

pushed branch u/gh-Bhatt21/wrong_output_timeit to correct this issue in src/sage/misc/sage_timeit.py

[Bhatt21]

Branch: u/gh-Bhatt21/wrong_output_timeit

[Bhatt21]

Commit: bc52a1c

[Bhatt21]

New commits:

bc52a1c

trac #15398 corrected sagetimeit grammar

[zimmermann6]

comment:7

looks good to me, thanks. Please could you tick the "needs review" button?

[zimmermann6]

Reviewer: Paul Zimmermann

[zimmermann6]

comment:9

please fill also the "Author" field.

[Bhatt21]

Author: gh-Bhatt21

[zimmermann6]

comment:11

looks good to me, thanks!

[jhpalmieri]

comment:12

Please enter your real name in the author field.

[Bhatt21]

Changed author from gh-Bhatt21 to Saket Bhatt

[Bhatt21]

comment:15

can someone tell me why this particular test pyflakes failing. It says time imported but not used but that is not the part of modified lines.

[fchapoton]

comment:16

as explained in the link on top of the patchbot report,

(https://wiki.sagemath.org/plugins#pyflakes)

pyflakes checks the modified files.

[Bhatt21]

comment:17

thanks. so I should get rid of this particular pyflakes error by making a new commit right?

[fchapoton]

comment:18

not necessarily, only if you can check that this removal does not break anything. It could be that "time" is imported from this file by another file, who knows..

[Bhatt21]

comment:19

true

[vbraun]

Changed branch from u/gh-Bhatt21/wrong_output_timeit to bc52a1c