profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/rchiodo/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.
Rich Chiodo rchiodo Microsoft Redmond, WA

rchiodo/vscode-python 1

Python extension for Visual Studio Code

aku911/piskel-react 0

Clone of piskel-app but using react

IanMatthewHuff/vscode-python 0

Python extension for Visual Studio Code

rchiodo/CMake 0

CMake, the cross-platform, open-source build system.

rchiodo/cpp-docs 0

C++ Documentation

rchiodo/ipykernel 0

IPython Kernel for Jupyter

rchiodo/ipython 0

Official repository for IPython itself. Other repos in the IPython organization contain things like the website, documentation builds, etc.

rchiodo/jupyterlab-commenting 0

Commenting and annotation for JupyterLab

rchiodo/jupyter_client 0

Jupyter protocol client APIs

rchiodo/jupyter_server 0

The backend—i.e. core services, APIs, and REST endpoints—to Jupyter web applications.

pull request commentmicrosoft/PTVS

Implement attach with debugpy

@int19h if you have a chance to look at this, that would be great. This is only the start of debugpy for attach though. I still need a way to attach if there's no python interpreter. Investigating using PyInstaller

rchiodo

comment created time in 4 hours

PR opened microsoft/PTVS

Fix wrapping for header labels

Fix this issue: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1355277

+15 -9

0 comment

4 changed files

pr created time in 4 hours

create barnchrchiodo/PTVS

branch : rchiodo/accessibility_2

created branch time in 4 hours

push eventrchiodo/PTVS

Rich Chiodo

commit sha d41c0e7a8039490abc666357e40a115b971660c7

Update to newer packages (#6633)

view details

Rich Chiodo

commit sha 1d6bb8b049d0ceb487289afba56b0c9e7b24520f

Two accessibility fixes (#6634)

view details

push time in 5 hours

push eventmicrosoft/PTVS

Rich Chiodo

commit sha 1d6bb8b049d0ceb487289afba56b0c9e7b24520f

Two accessibility fixes (#6634)

view details

push time in 5 hours

PR merged microsoft/PTVS

Reviewers
Two accessibility fixes

Fixes

  • https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1351212
  • https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1351247
+6 -4

2 comments

3 changed files

rchiodo

pr closed time in 5 hours

push eventmicrosoft/PTVS

Rich Chiodo

commit sha d41c0e7a8039490abc666357e40a115b971660c7

Update to newer packages (#6633)

view details

push time in 5 hours

PR merged microsoft/PTVS

Reviewers
Update to newer packages

Update to newer packages for imports

+48 -29

1 comment

4 changed files

rchiodo

pr closed time in 5 hours

PR opened microsoft/PTVS

Two accessibility fixes

Fixes

  • https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1351212
  • https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1351247
+6 -4

0 comment

3 changed files

pr created time in 5 hours

create barnchrchiodo/PTVS

branch : rchiodo/accessibility

created branch time in 6 hours

PullRequestReviewEvent

Pull request review commentmicrosoft/vscode-jupyter

Revert work around for not being able to install ipykernel

 export class PythonInstaller implements IPythonInstaller {             this.interpreterPackages.trackPackages(resource);         }         let action: 'installed' | 'failed' | 'disabled' | 'ignored' = 'installed';-        if (env.remoteName && reInstallAndUpdate) {-            // Temporary work around for https://github.com/microsoft/vscode-jupyter/issues/6896-            traceWarning(

Is the bug still active? Wondering how we make sure to fix this for real before next release.

DonJayamanne

comment created time in 7 hours

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentmicrosoft/vscode-jupyter

Minimize unnecessary message deserialization

 export class IPyWidgetMessageDispatcher implements IIPyWidgetMessageDispatcher {         let message;          if (!this.isUsingIPyWidgets) {-            if (!message) {+            // Lets deserialize only if we know we have a potential case+            // where this message contains some data we're interested in.+            let mustDeserialize = false;+            if (typeof data === 'string') {+                mustDeserialize = data.includes(WIDGET_MIMETYPE) || data.includes(Identifiers.DefaultCommTarget);

Misread the section below. THis is just like the previous change.

DonJayamanne

comment created time in 8 hours

PullRequestReviewEvent

issue closedmicrosoft/PTVS

Failed to create "GitHub->cookiecutter-flask/cookiecutter-flask" template in Cookiecutter.

Describe the bug

image

Steps to Reproduce

  1. Launch VS
  2. Click "View->Cookiecutter Explorer"
  3. Select "GitHub->cookiecutter-flask/cookiecutter-flask" to click "Next"
  4. Wait for load successfully and then click "Create and Open Folder"

Expected behavior

It should create successfully.

Additional context and screenshots

Error occurs when create template.

image

Error detail info

----- Creating files using template 'cookiecutter-flask/cookiecutter-flask' -----
DEBUG main.py: context_file is C:\Users\vting\.cookiecutters\cookiecutter-flask\cookiecutter.json
DEBUG generate.py: Context generated is {'cookiecutter': OrderedDict([('full_name', ''), ('email', ''), ('github_username', ''), ('project_name', ''), ('app_name', ''), ('project_short_description', ''), ('use_pipenv', ''), ('python_version', ''), ('node_version', ''), ('use_heroku', '')])}
DEBUG utils.py: Making sure path exists: C:\Users\vting/.cookiecutter_replay/
DEBUG find.py: Searching C:\Users\vting\.cookiecutters\cookiecutter-flask for the project template.
DEBUG find.py: The project template appears to be C:\Users\vting\.cookiecutters\cookiecutter-flask\{{cookiecutter.app_name}}
DEBUG generate.py: Generating project from C:\Users\vting\.cookiecutters\cookiecutter-flask\{{cookiecutter.app_name}}...
DEBUG generate.py: Rendered dir  must exist in output_dir C:\Users\vting\AppData\Local\Temp\85d45923-8a2b-44be-97f7-3883dda14121
Traceback (most recent call last):
  File "c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\cookiecutter\cookiecutter_run.py", line 46, in <module>
    sys.exit(int(main() or 0))
  File "c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\cookiecutter\cookiecutter_run.py", line 37, in main
    cookiecutter(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\main.py", line 140, in cookiecutter
    return generate_files(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\generate.py", line 264, in generate_files
    project_dir = render_and_create_dir(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\generate.py", line 211, in render_and_create_dir
    raise OutputDirExistsException(msg)
cookiecutter.exceptions.OutputDirExistsException: Error: "C:\Users\vting\AppData\Local\Temp\85d45923-8a2b-44be-97f7-3883dda14121" directory already exists
C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\scripts\python.exe returned an exit code of 1.
----- Failed to create files using template 'cookiecutter-flask/cookiecutter-flask' -----

closed time in 9 hours

ttSpace

issue commentmicrosoft/PTVS

Failed to create "GitHub->cookiecutter-flask/cookiecutter-flask" template in Cookiecutter.

Yeah I get this is I do this more than once. The user has to specify a new directory here before creating:

image

Or delete the old folder.

ttSpace

comment created time in 9 hours

PullRequestReviewEvent

Pull request review commentmicrosoft/vscode-jupyter

Minimize unnecessary message deserialization

 export class IPyWidgetMessageDispatcher implements IIPyWidgetMessageDispatcher {         let message;          if (!this.isUsingIPyWidgets) {-            if (!message) {+            // Lets deserialize only if we know we have a potential case+            // where this message contains some data we're interested in.+            let mustDeserialize = false;+            if (typeof data === 'string') {+                mustDeserialize = data.includes(WIDGET_MIMETYPE) || data.includes(Identifiers.DefaultCommTarget);

This is a pretty big assumption. How do you know that all messages for a widget will fit into these subcategories? And what about when we start supporting synapse?

What's the perf improvement of this change? Seems risky to me.

DonJayamanne

comment created time in 9 hours

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentmicrosoft/vscode-python

Fix interactive window diagnostics position

 export class InteractiveConcatTextDocument implements IConcatTextDocument {             }
         });
 
-        this._updateConcat();
-        this._updateInput();
+        /** for testing only */

Should this still be here?

rebornix

comment created time in 10 hours

PullRequestReviewEvent

PR opened microsoft/PTVS

Update to newer packages

Update to newer packages for imports

+48 -29

0 comment

4 changed files

pr created time in 11 hours

create barnchrchiodo/PTVS

branch : rchiodo/check_issues

created branch time in 11 hours