Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rchiodo
Copy link

@rchiodo rchiodo commented Aug 10, 2020

Finally got this to repro randomly (Direct Kernel disconnected). It seems this can occur if the editor or interactive window is holding onto a disposed notebook when multiple copies of the same file are opened and you close one.

Multiple copies of an editor opening is hard to repro though.

@rchiodo rchiodo added the no-changelog No news entry required label Aug 10, 2020
@rchiodo rchiodo self-assigned this Aug 10, 2020
@codecov-commenter
Copy link

codecov-commenter commented Aug 10, 2020

Codecov Report

Merging #13373 into master will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #13373      +/-   ##
==========================================
- Coverage   59.71%   59.68%   -0.04%     
==========================================
  Files         670      670              
  Lines       37338    37342       +4     
  Branches     5307     5308       +1     
==========================================
- Hits        22298    22289       -9     
- Misses      13905    13915      +10     
- Partials     1135     1138       +3     
Impacted Files Coverage Δ
.../datascience/interactive-common/interactiveBase.ts 5.49% <0.00%> (-0.04%) ⬇️
src/client/common/utils/platform.ts 64.70% <0.00%> (-11.77%) ⬇️
src/client/datascience/crossProcessLock.ts 79.41% <0.00%> (-11.77%) ⬇️
src/client/linters/pydocstyle.ts 86.66% <0.00%> (-2.23%) ⬇️
src/client/datascience/debugLocationTracker.ts 76.56% <0.00%> (-1.57%) ⬇️
src/client/common/process/proc.ts 14.49% <0.00%> (-0.73%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e62824...0ca193e. Read the comment docs.


protected async ensureConnectionAndNotebook(): Promise<void> {
// Start over if we somehow end up with a disposed notebook.
if (this._notebook && this._notebook.disposed) {
Copy link
Member

@IanMatthewHuff IanMatthewHuff Aug 10, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seem good. Just reset if the notebook was disposed. So it was one of those situations where a ipynb was opened twice?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was trying to validate a fix you made about causing files to be locked out and got the same file to open multiple times. When I tried to run it, I got this error.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rchiodo rchiodo merged commit 0441636 into master Aug 10, 2020
@rchiodo rchiodo deleted the rchiodo/notebook_disposed branch August 10, 2020 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants