Skip to content

Commit

Permalink
Remove var inspect log messages (#12180)
Browse files Browse the repository at this point in the history
They have served their purpose
  • Loading branch information
roblourens authored Nov 27, 2022
1 parent 1ca6ee1 commit 09d4d5e
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/kernels/variables/kernelVariables.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import { Identifiers, PYTHON_LANGUAGE } from '../../platform/common/constants';
import { Experiments } from '../../platform/common/experiments/groups';
import { IConfigurationService, IDisposableRegistry, IExperimentService } from '../../platform/common/types';
import { createDeferred } from '../../platform/common/utils/async';
import { traceVerbose } from '../../platform/logging';
import { getKernelConnectionLanguage, isPythonKernelConnection } from '../helpers';
import { IKernel, IKernelConnectionSession, IKernelProvider } from '../types';
import {
Expand Down Expand Up @@ -344,14 +343,12 @@ export class KernelVariables implements IJupyterVariables {
): Promise<IJupyterVariable> {
let result = { ...targetVariable };
if (!kernel.disposed && kernel.session) {
traceVerbose(`Inspecting '${targetVariable.name}'`);
const output = await this.inspect(kernel.session, targetVariable.name, 0, token);

// Should be a text/plain inside of it (at least IPython does this)
if (output && output.hasOwnProperty('text/plain')) {
// eslint-disable-next-line @typescript-eslint/no-explicit-any
const text = (output as any)['text/plain'].toString() as string;
traceVerbose(`Inspected '${targetVariable.name}' and got ${text.length} characters`);

// Parse into bits
const type = TypeRegex.exec(text);
Expand Down

0 comments on commit 09d4d5e

Please sign in to comment.