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

[api][commands] Open new window and a file #123615

Open
roblourens opened this issue May 12, 2021 · 6 comments
Open

[api][commands] Open new window and a file #123615

roblourens opened this issue May 12, 2021 · 6 comments
Assignees
Labels
api feature-request Request for new features or functionality
Milestone

Comments

@roblourens
Copy link
Member

roblourens commented May 12, 2021

The vscode.newWindow command takes a remoteAuthority argument and opens a window on that authority. The Azure ML extension needs to open a window on a certain remote, then open a file on that remote. They do this by adding extra data to the remoteAuthority. This works but a downside is that you end up with extra entries in recently opened, since these windows have different remote authorities and are considered distinct. A better way to do this would be to allow passing in extra arguments that go to the resolver. Maybe they would be a fragment or query on the opened workspace folder URI, I don't know whether we ever do that.

@aeschli
Copy link
Contributor

aeschli commented May 12, 2021

Maybe we should look into extra arguments to the newWindow commands? Files to open?

@roblourens
Copy link
Member Author

I was just thinking of custom arguments for an extension, but yeah I guess that would work as an argument handled by vscode too. But the extension uses showNotebookDocument and might pass some other options to it too.

@SiddhanthUnnithan do you pass any extra options to showNotebookDocument or would a generic "open the file" option work for your case?

@SiddhanthUnnithan
Copy link

I was just thinking of custom arguments for an extension, but yeah I guess that would work as an argument handled by vscode too. But the extension uses showNotebookDocument and might pass some other options to it too.

@SiddhanthUnnithan do you pass any extra options to showNotebookDocument or would a generic "open the file" option work for your case?

Currently we pass "preserveFocus: False" and "preview: False" as arguments.

@aeschli
Copy link
Contributor

aeschli commented Oct 18, 2021

Is this still wanted?

@aeschli aeschli added the api label Oct 18, 2021
@aeschli aeschli changed the title Open remote window with arguments Open new window and a file Oct 18, 2021
@aeschli aeschli added the feature-request Request for new features or functionality label Oct 18, 2021
@aeschli aeschli added this to the Backlog Candidates milestone Oct 18, 2021
@shsuman
Copy link
Member

shsuman commented Oct 21, 2021

Hi Martin,
This is a really important ask for Azure ML - Remote. Currently, we have to pass arguments in the URI itself using the following format: vscode-remote://amext+arg1={arg1}+arg2={arg2}/folderPath. This has led to multiple problems for us in the past, the most recent one being that the workspace trust prompt. Since our URI is only unique for a session (because of some telemetry tracking ids), the workspace trust prompt is shown every time we open a remote window on a Compute instance even if its the same folder.

@aeschli aeschli changed the title Open new window and a file [api][commands] Open new window and a file Oct 29, 2021
@aeschli aeschli modified the milestones: Backlog Candidates, On Deck Oct 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

5 participants
@roblourens @SiddhanthUnnithan @aeschli @shsuman and others