Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How should extension inspector scripts be created for new languages? #202

Open
gerph opened this issue Apr 4, 2021 · 3 comments
Open

How should extension inspector scripts be created for new languages? #202

gerph opened this issue Apr 4, 2021 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@gerph
Copy link

gerph commented Apr 4, 2021

Hiya,

I've created a new kernel for a pair of 'languages' that I want to use. I want to use variable inspector with these new languages, but the only way I can see to do that is to directly modify the commands that are in the inspectorscripts.ts file and provide new parser code for this. I cannot see a way to do this without essentially forking this project and adding the necessary code to my own fork.

Is there a way to extend the languages supported through an extension instead of by forking the project and directly modifying it?

@lckr
Copy link
Collaborator

lckr commented Apr 10, 2021

Hey, you're right, at the moment one would have to modify the code in inspectorscripts.ts. A better approach would be to add language scripts in a separate directory. I will investigate if this would work with the way the packge is build and distributed at the moment.

@lckr
Copy link
Collaborator

lckr commented Apr 10, 2021

If someone has time to tackle this, one would have to to the following:

  • Create separate directory for language scripts
  • Parse existing scripts at start up
  • If kernel with language L is connected to the extension: Check if there is an available script that we can use.

@lckr lckr added the help wanted Extra attention is needed label Apr 10, 2021
@yitzchak
Copy link

yitzchak commented Jun 8, 2021

Why not create your own custom COMM target so the the support can be contained in the kernel? Inspection would be done via COMM messages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants