Skip to content
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

time magic: shorten unnecessary output on windows #2961

Merged
merged 1 commit into from Mar 10, 2013
Merged

time magic: shorten unnecessary output on windows #2961

merged 1 commit into from Mar 10, 2013

Conversation

jankatins
Copy link
Contributor

On windows the detailed info (i.e. cpu time) is not available and
always reported as zero. The rest of the information is therefore
redundant with the next print (wall time) and can be omitted.

On windows the detailed info (i.e. cpu time) is not available and
always reported as zero. The rest of the information is therefore
redundant with the next print (wall time) and can be omitted.
@jankatins
Copy link
Contributor Author

I'm not sure what the value of sys and user time is. Maybe even remove this output in default mode and only show it on "-v"?

@minrk
Copy link
Member

minrk commented Feb 21, 2013

I'm not sure what the value of sys and user time is.

The %time magic is meant to mimic the unix time command, which does this breakdown. I don't think I would change it on platforms where it makes sense.

minrk added a commit that referenced this pull request Mar 10, 2013
%time magic: shorten unnecessary output on windows

On windows the detailed info (i.e. cpu time) is not available and
always reported as zero. The rest of the information is therefore
redundant with the next print (wall time) and can be omitted.
@minrk minrk merged commit 89ca332 into ipython:master Mar 10, 2013
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
%time magic: shorten unnecessary output on windows

On windows the detailed info (i.e. cpu time) is not available and
always reported as zero. The rest of the information is therefore
redundant with the next print (wall time) and can be omitted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants