profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/karthiknadig/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Karthik Nadig karthiknadig @Microsoft Redmond Developer at Microsoft, working on VS Code Python and Python Debugger.

karthiknadig/ArduinoFIS 4

Matlab FIS files to *.c converter.

karthiknadig/papers-we-love 1

Papers from the computer science community to read and discuss.

karthiknadig/corefx 0

This repo contains the .NET Core foundational libraries, called CoreFX. It includes classes for collections, file systems, console, XML, async and many others.

karthiknadig/cpython 0

The Python programming language

karthiknadig/debugpy 0

An implementation of the Debug Adapter Protocol for Python

karthiknadig/DirectXMath 0

DirectXMath is an all inline SIMD C++ linear algebra library for use in games and graphics apps

karthiknadig/django_barcode_sample 0

Sample Django app that generates SVG barcode images using EAN13 or EAN8 format.

karthiknadig/FileMeta 0

Enable Explorer in Vista, Windows 7 and later to see, edit and search on tags and other metadata for any file type

issue commentmicrosoft/vscode-python

Test discovery failure

@dcavanagh Can you share the function and its params? also are you using any pytest plugins?

dcavanagh

comment created time in 2 hours

PullRequestReviewEvent

delete branch karthiknadig/vscode-python

delete branch : release-candidate

delete time in 6 hours

delete branch karthiknadig/vscode-python

delete branch : main-version

delete time in 6 hours

push eventkarthiknadig/vscode-python

Kim-Adeline Miguel

commit sha bdcfe1aafa05c756c88d74e8d2f49b9fffd6a639

Activate extension on walkthrough (#16808) * Activate extension on walkthrough * Activate for DS walkthrough as well

view details

Karthik Nadig

commit sha 92ef490c1aacc13404f35a80ab95d9fcb6884c15

Remove MPLS as enum from package.json (#16823)

view details

Jasleen Sondhi

commit sha c72e4fe95b3ef4c4f03f69c2c5e0b10082f890e0

Changed "python.languageServer" to "Pylance" as per Issue #13007 (#16804) * Changes "python.languageServer" to "Pylance" * Update news/.vscode/settings.json Changed "python.languageServer" to "Pylance" Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com> * Updated CHANELOG with a new news entry * Removed the changes done Had added a manual entry which was removed as suggested. * Changed "python.languageServer" to "Pylance" Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>

view details

Luciana Abud

commit sha 67cd0f306185bd67261d9e4800a43f41fc7d79fc

Fixes for getting started page (#16820) * Add color themed screenshots for interactive walkthrough * Add color themed screenshots for new notebook walkthrough * Fix install Anaconda button * Fix tutorial link on data science walkthrough

view details

Kim-Adeline Miguel

commit sha 442c6e41494b8c20325d94b29f6b518347e8ea12

Remove myself from the list of reviewers (#16832)

view details

Brett Cannon

commit sha 324ef4c8760495fa6ac23b3345509b3223f3b8d0

Take Paula out of the reviewer rotation

view details

dependabot[bot]

commit sha e7fa22636693730133239ad1dfc97db03181df81

Bump isort from 5.9.2 to 5.9.3 (#16826) Bumps [isort](https://github.com/pycqa/isort) from 5.9.2 to 5.9.3. - [Release notes](https://github.com/pycqa/isort/releases) - [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md) - [Commits](https://github.com/pycqa/isort/compare/5.9.2...5.9.3) --- updated-dependencies: - dependency-name: isort dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

view details

Peng Lyu

commit sha 6edad1670951d0bacf403096cd584e8c7ee63c77

Fix interactive window diagnostics position (#16838) * Fix interactive window diagnostics position. * resolve review.

view details

Karthik Nadig

commit sha 2e9e8ab789f8a556224b621080de5da29a1a72ed

Release candidate (#16847) * Update version * Update change log * Update TPN

view details

Karthik Nadig

commit sha 57d8cf221d65fca16de4efce5dadad60375309fb

Update version (#16850)

view details

Karthik Nadig

commit sha 843ff4b5c36b52c6782b380a6a66c510223d1e5f

migrating User PR to test branch: Fix broken unittest test discovery (#16703) * Fix broken unittest test discovery Test discovery with unittest was not working at all. This is now fixed with a more reliable method. (Issue 16593) * Fix typo Somehow I managed to typo such a simple change. Co-authored-by: Julian Edwards <bigjools@gmail.com>

view details

Karthik Nadig

commit sha 5819912b3762fa8322709d22132a4d9e6903e97c

Remove code lenses used in tests (#16200)

view details

Karthik Nadig

commit sha e72ccc04cff15c2db4f7c620f0c678d2ae5f155a

Add test view with a welcome button (#16215) * Add test view with a welcome button * Register registerTestController only when available.

view details

Karthik Nadig

commit sha 6ee0305ca68a9662ec3dc278753c977ed761aa0d

Remove nosetest support (#16217) * Remove nosetest support * Fix enablement tests * Update src/client/testing/common/xUnitParser.ts Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com> Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>

view details

Karthik Nadig

commit sha 92b855f3f2b63f728ad815ca3de28479b797d4f5

Remove navigation commands (#16276)

view details

Karthik Nadig

commit sha 34ddf557ed777ebb1b4982496d8356d3be7e2073

Removing view test ui and view test output commands (#16298)

view details

Karthik Nadig

commit sha 2be18de96d9a9676463fbb1de9142a8d68041e86

Remove test picker (#16300)

view details

Karthik Nadig

commit sha 8e3d0c8700b963f5b7bdc95c6107444cf2331f77

Remove more test commands (#16314) * Remove select and run commands * Remove commands from package json

view details

Karthik Nadig

commit sha a9c7c1b9beebb2157e6e8a9599703efde2615c56

Remove more test commands (#16315)

view details

Karthik Nadig

commit sha 8da632d2fb44eb410658c7d7bb120aa1512b671b

Remove testing dependency on language server (#16319)

view details

push time in 6 hours

push eventmicrosoft/vscode-python

Karthik Nadig

commit sha 57d8cf221d65fca16de4efce5dadad60375309fb

Update version (#16850)

view details

push time in 6 hours

PR merged microsoft/vscode-python

Update main branch version skip news
+2 -2

0 comment

2 changed files

karthiknadig

pr closed time in 6 hours

PR opened microsoft/vscode-python

Update main branch version skip news
+2 -2

0 comment

2 changed files

pr created time in 6 hours

create barnchkarthiknadig/vscode-python

branch : main-version

created branch time in 6 hours

create barnchmicrosoft/vscode-python

branch : release-2021.08

created branch time in 6 hours

push eventmicrosoft/vscode-python

Karthik Nadig

commit sha 2e9e8ab789f8a556224b621080de5da29a1a72ed

Release candidate (#16847) * Update version * Update change log * Update TPN

view details

push time in 6 hours

PR merged microsoft/vscode-python

Release candidate skip news
+78 -10

0 comment

8 changed files

karthiknadig

pr closed time in 6 hours

PR opened microsoft/vscode-python

Release candidate skip news
+78 -10

0 comment

8 changed files

pr created time in 7 hours

create barnchkarthiknadig/vscode-python

branch : release-candidate

created branch time in 7 hours

issue commentmicrosoft/vscode-python

While debugging hovering on array doesn't show the indexed value

Thanks for the details @weinand .

mrx23dot

comment created time in 8 hours

delete branch karthiknadig/vscode-python

delete branch : mergeback

delete time in 9 hours

delete branch karthiknadig/vscode-python

delete branch : remove-mpls

delete time in 9 hours

delete branch karthiknadig/vscode-python

delete branch : testing1

delete time in 9 hours

push eventkarthiknadig/vscode-python

Karthik Nadig

commit sha eae6b4834d7039b14ef97050a9da7677555a2c77

Mergeback changes from point release to main (#16809) * Fix 'Cannot read property 'resolveEnv' of undefined' error (#16677) * Cherry pick fixes into release (#16686) * Bump actions/setup-node from 2.1.5 to 2.2.0 (#16592) Bumps [actions/setup-node](https://github.com/actions/setup-node) from 2.1.5 to 2.2.0. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](https://github.com/actions/setup-node/compare/v2.1.5...v2.2.0) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add support for starting TensorBoard session with a remote log directory via URL (#16477) * Add support for remote logdirs * Add missing keys * Fix typo * Bump isort from 5.8.0 to 5.9.2 (#16636) Bumps [isort](https://github.com/pycqa/isort) from 5.8.0 to 5.9.2. - [Release notes](https://github.com/pycqa/isort/releases) - [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md) - [Commits](https://github.com/pycqa/isort/compare/5.8.0...5.9.2) --- updated-dependencies: - dependency-name: isort dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Make getInterpreters() API faster for subsequent calls (#16674) * Change the way auto-selection works (#16644) * New comparison logic * Add experiment group * Register and call it * Add service registry tests * Add interpreter selector unit tests * Add comparison unit tests * Add intepreter selector test * News file * Adjust comments * Reuse getSortName * Add new auto-selection logic * Add tests for getEnvTypeHeuristic * Move persistent store initialization back out * Update tests Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joyce Er <joyceerhuiling@gmail.com> Co-authored-by: Kartik Raj <karraj@microsoft.com> Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com> * Change version for release (#16722) * Ensure we block on autoselection when no interpreter is explictly set by user (#16723) * Ensure we block on autoselection when no interpreter is explictly set by user * Added tests * News entry * Update change log for release. (#16731) * Fix autoselection when opening a python file directly (#16733) * Fix autoselection when opening a python file directly * Update changelog * Add tests * Ignore cache when querying for interpreters during auto-selection (#16734) * Ignore cache when getting envs for autoselection * Don't call autoSelectInterpreter twice * Update debugger via point release (#16746) * Update version * Update change log * Update wheels to 3.9 (#16745) * Point release with debugpy fix (#16776) Co-authored-by: Kartik Raj <karraj@microsoft.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joyce Er <joyceerhuiling@gmail.com> Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>

view details

Kim-Adeline Miguel

commit sha bdcfe1aafa05c756c88d74e8d2f49b9fffd6a639

Activate extension on walkthrough (#16808) * Activate extension on walkthrough * Activate for DS walkthrough as well

view details

Karthik Nadig

commit sha 92ef490c1aacc13404f35a80ab95d9fcb6884c15

Remove MPLS as enum from package.json (#16823)

view details

Jasleen Sondhi

commit sha c72e4fe95b3ef4c4f03f69c2c5e0b10082f890e0

Changed "python.languageServer" to "Pylance" as per Issue #13007 (#16804) * Changes "python.languageServer" to "Pylance" * Update news/.vscode/settings.json Changed "python.languageServer" to "Pylance" Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com> * Updated CHANELOG with a new news entry * Removed the changes done Had added a manual entry which was removed as suggested. * Changed "python.languageServer" to "Pylance" Co-authored-by: Kim-Adeline Miguel <51720070+kimadeline@users.noreply.github.com>

view details

Luciana Abud

commit sha 67cd0f306185bd67261d9e4800a43f41fc7d79fc

Fixes for getting started page (#16820) * Add color themed screenshots for interactive walkthrough * Add color themed screenshots for new notebook walkthrough * Fix install Anaconda button * Fix tutorial link on data science walkthrough

view details

Kim-Adeline Miguel

commit sha 442c6e41494b8c20325d94b29f6b518347e8ea12

Remove myself from the list of reviewers (#16832)

view details

Brett Cannon

commit sha 324ef4c8760495fa6ac23b3345509b3223f3b8d0

Take Paula out of the reviewer rotation

view details

dependabot[bot]

commit sha e7fa22636693730133239ad1dfc97db03181df81

Bump isort from 5.9.2 to 5.9.3 (#16826) Bumps [isort](https://github.com/pycqa/isort) from 5.9.2 to 5.9.3. - [Release notes](https://github.com/pycqa/isort/releases) - [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md) - [Commits](https://github.com/pycqa/isort/compare/5.9.2...5.9.3) --- updated-dependencies: - dependency-name: isort dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

view details

Peng Lyu

commit sha 6edad1670951d0bacf403096cd584e8c7ee63c77

Fix interactive window diagnostics position (#16838) * Fix interactive window diagnostics position. * resolve review.

view details

push time in 9 hours

push eventmicrosoft/vscode-python

Peng Lyu

commit sha 6edad1670951d0bacf403096cd584e8c7ee63c77

Fix interactive window diagnostics position (#16838) * Fix interactive window diagnostics position. * resolve review.

view details

push time in 9 hours

delete branch microsoft/vscode-python

delete branch : dev/rebornix/fix-iw-problems-position

delete time in 9 hours

PR merged microsoft/vscode-python

Reviewers
Fix interactive window diagnostics position skip news skip tests

Fix interactive window concat document position offset bug like below

image

With the PR

image

+19 -19

2 comments

1 changed file

rebornix

pr closed time in 9 hours

issue commentmicrosoft/vscode-python

Shift-enter opens terminal panel when REPL is in editor area

Thanks for the bug report! We investigate issues in order based on priority and severity, which includes the impact it has on your ability to use the extension to do productive work, and the number of people affected. If other users come forward and leave a comment demonstrating they are seeing/reproducing the problem then we will raise this issue's priority.

If you think your issue is more of a question or configuration problem rather than a bug, please ask on Stack Overflow with the visual-studio-code and python labels for help.

Thanks for your understanding and patience!

arjunguha

comment created time in 10 hours

issue commentmicrosoft/vscode-python

While debugging hovering on array doesn't show the indexed value

It looks like the debugger does not receive evaluate request for the whole thing. Instead the debugger either gets newKeys or invested_capital (also, the string does not seem to be complete in this case.

evaluate request for the dict newKeys:

D+00010.422: Client[1] --> {
                 "seq": 15,
                 "type": "request",
                 "command": "evaluate",
                 "arguments": {
                     "expression": "newKeys",
                     "frameId": 2,
                     "context": "hover"
                 }
             }

evaluate request for the key "invested_capital:

Not sure why the ending quote is missed in the evaluate request.

D+00017.110: Client[1] --> {
                 "seq": 17,
                 "type": "request",
                 "command": "evaluate",
                 "arguments": {
                     "expression": "\"invested_capital",
                     "frameId": 2,
                     "context": "hover"
                 }
             }

/cc @weinand any suggestions on where this issue should be handled? I believe hover on [ or ] in should send the entire newKeys['invested_capital'] for evaluation.

mrx23dot

comment created time in 10 hours

issue commentmicrosoft/vscode-python

windows run installPythonLibs will fail with "Can not combine '--user' and '--target'"

Thanks for identifying the problem. Please feel free to create a PR with the fix.

slightc

comment created time in 11 hours

Pull request review commentmicrosoft/vscode-python

Remove start page and its build dependencies

 const ts = require('gulp-typescript'); const spawn = require('cross-spawn'); const path = require('path'); const del = require('del');-const fs = require('fs-extra');

Good Catch :)

kimadeline

comment created time in 2 days

PullRequestReviewEvent

Pull request review commentmicrosoft/vscode-python

Remove start page and its build dependencies

 gulp.task('precommit', (done) => run({ exitOnError: true, mode: 'staged' }, done  gulp.task('output:clean', () => del(['coverage'])); -gulp.task('clean:cleanExceptTests', () => del(['clean:vsix', 'out/client', 'out/startPage-ui', 'out/server']));+gulp.task('clean:cleanExceptTests', () => del(['clean:vsix', 'out/client', 'out/server']));

What is out/server for? I think this might have been for debug adapter server that was missed when we removed it.

kimadeline

comment created time in 2 days

PullRequestReviewEvent

push eventmicrosoft/vscode-python

dependabot[bot]

commit sha e7fa22636693730133239ad1dfc97db03181df81

Bump isort from 5.9.2 to 5.9.3 (#16826) Bumps [isort](https://github.com/pycqa/isort) from 5.9.2 to 5.9.3. - [Release notes](https://github.com/pycqa/isort/releases) - [Changelog](https://github.com/PyCQA/isort/blob/main/CHANGELOG.md) - [Commits](https://github.com/pycqa/isort/compare/5.9.2...5.9.3) --- updated-dependencies: - dependency-name: isort dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

view details

push time in 12 hours