Skip to content

Commit

Permalink
Show active interpreter controller only for local
Browse files Browse the repository at this point in the history
  • Loading branch information
DonJayamanne committed Sep 7, 2021
1 parent f08a71f commit ec99fcb
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/client/datascience/notebook/notebookControllerManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -288,7 +288,11 @@ export class NotebookControllerManager implements INotebookControllerManager, IE
traceInfoIf(isCI, `Clear controller mapping for ${document.uri.toString()}`);
const loadControllersPromise = this.loadNotebookControllers();

if (isPythonNotebook(getNotebookMetadata(document)) && this.extensionChecker.isPythonExtensionInstalled) {
if (
isPythonNotebook(getNotebookMetadata(document)) &&
this.extensionChecker.isPythonExtensionInstalled &&
this.isLocalLaunch
) {
// If we know we're dealing with a Python notebook, load the active interpreter as a kernel asap.
this.createActiveInterpreterController(JupyterNotebookView).catch(noop);
}
Expand Down

0 comments on commit ec99fcb

Please sign in to comment.