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

Update execution_count before running cell #119

Merged
merged 1 commit into from
Feb 21, 2017

Conversation

wilywampa
Copy link
Contributor

This change ensures the execution count displayed next to code is correct even if the execution count has been updated in another session talking to the same kernel.

Ensure the execution count displayed next to the code is correct by
processing any IOPub messages between pressing enter and running the
code. The execution count should always be updated after an
execute_input message even if the message came from a session whose
output wouldn't be included because it is still the execution count of
this session's kernel.
@wilywampa wilywampa closed this Feb 18, 2017
@wilywampa wilywampa reopened this Feb 18, 2017
@wilywampa
Copy link
Contributor Author

Any thoughts on why this is failing? I don't see any difference with or without this change when I run the test commands manually. Also, it looks like it's only failing for Python 3.6 which is odd.

@takluyver
Copy link
Member

I reran the failing test and it passed; I think it was just a random failure. This looks reasonable to me.

@takluyver takluyver added this to the 5.2 milestone Feb 20, 2017
@minrk minrk merged commit e1af67b into jupyter:master Feb 21, 2017
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.

3 participants