New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add timer unit argument for output time granularity specification #98

Merged
merged 1 commit into from Oct 24, 2017

Conversation

Projects
None yet
3 participants
@cshaley

cshaley commented Aug 30, 2017

Example test run:
In:

def m():
    return [0]*10**8

In:
%lprun -u 1e-3 -f m m()

Out:
Timer unit: 0.001 s

Total time: 0.363548 s
File:
Function: m at line 1

Line # Hits Time Per Hit % Time Line Contents
==============================================================
1 def m():
2 1 363.5 363.5 100.0 return [0]*10**8

Solution to #97

Show outdated Hide outdated line_profiler.py
@caethan

This comment has been minimized.

Show comment
Hide comment
@caethan

caethan Oct 20, 2017

Collaborator

I like it! Can you update the docs to specify that the -u option takes the value in seconds?

Collaborator

caethan commented Oct 20, 2017

I like it! Can you update the docs to specify that the -u option takes the value in seconds?

@cshaley

This comment has been minimized.

Show comment
Hide comment
@cshaley

cshaley Oct 22, 2017

@caethan Just the readme? Those are the only docs I can find, and many of the arguments are not documented there either...

cshaley commented Oct 22, 2017

@caethan Just the readme? Those are the only docs I can find, and many of the arguments are not documented there either...

@caethan

This comment has been minimized.

Show comment
Hide comment
@caethan

caethan Oct 23, 2017

Collaborator

I just meant the docstring.

Collaborator

caethan commented Oct 23, 2017

I just meant the docstring.

@cshaley

This comment has been minimized.

Show comment
Hide comment
@cshaley

cshaley commented Oct 23, 2017

Ok done.

@caethan caethan merged commit 4e3ce59 into rkern:master Oct 24, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment