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

Improve viewing large text output #209158

Closed
Tracked by #215397
rebornix opened this issue Mar 29, 2024 · 4 comments
Closed
Tracked by #215397

Improve viewing large text output #209158

rebornix opened this issue Mar 29, 2024 · 4 comments
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@rebornix
Copy link
Member

rebornix commented Mar 29, 2024

Type: Feature Request

While experimenting with LLMs in Jupyter notebook, I often need to validate the text output (custom prompt, documents retrivied, etc). I like the fact that it's rendered in a scrollabe output and I can search text in it. However in order to read through them line by line, I found myself repeatedly:

  • Click in the output
  • Cmd+A
  • Cmd+C
  • Cmd+N to open a new text editor
  • Cmd+V

Before we have the scrollable output support, we render a command "open in text editor" in the output which can be really convinient, but I'm not sure I want to sacrifice scrollable output for it though.

VS Code version: Code - Insiders 1.88.0-insider (aa25485, 2024-03-28T22:10:25.599Z)
OS version: Windows_NT x64 10.0.22631
Modes:

rebornix added a commit that referenced this issue Jun 23, 2024
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Jun 24, 2024
@VSCodeTriageBot VSCodeTriageBot added this to the June 2024 milestone Jun 24, 2024
@rebornix
Copy link
Member Author

We can now right click on a large text output and open directly in a text editor

image

@rebornix rebornix added the verification-needed Verification of issue is requested label Jun 24, 2024
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jun 25, 2024
@rzhao271 rzhao271 added verified Verification succeeded and removed verified Verification succeeded labels Jun 25, 2024
@rzhao271
Copy link
Contributor

I encountered microsoft/vscode-jupyter#15639 but I believe it's due to my setup. Taking the verified label off for someone else to verify.

@rzhao271
Copy link
Contributor

I'm confused on how the feature works.
I have set up a Conda environment on my Windows machine and run a notebook with a single cell:

for i in range(1000):
    print(5 * i)

I see the following behaviour:
If I run the notebook, close the notebook, and open it again, then I can right-click and view the output in a text editor. ✅
If I change the 5 to a 6, run the notebook, and try to right-click and view the output in a text editor, nothing happens. 🐛

@rzhao271 rzhao271 reopened this Jun 25, 2024
@VSCodeTriageBot VSCodeTriageBot removed the insiders-released Patch has been released in VS Code Insiders label Jun 25, 2024
@rzhao271 rzhao271 added verification-found Issue verification failed insiders-released Patch has been released in VS Code Insiders labels Jun 25, 2024
@amunger amunger assigned rebornix and unassigned amunger Jun 25, 2024
bricefriha pushed a commit to bricefriha/vscode that referenced this issue Jun 26, 2024
…to Scrollable text output context menu. (microsoft#216920)

* Fix microsoft#209158. Add Copy Output and Open Output in Text Editor to Scrollable text output context menu.

* Context menu should be triggered on container

* fix naming
@rebornix rebornix removed the verification-found Issue verification failed label Jun 26, 2024
@rebornix
Copy link
Member Author

Should be fixed now.

@rzhao271 rzhao271 added feature-request Request for new features or functionality verified Verification succeeded labels Jun 26, 2024
aaronchucarroll pushed a commit to aaronchucarroll/vscode that referenced this issue Jul 10, 2024
…to Scrollable text output context menu. (microsoft#216920)

* Fix microsoft#209158. Add Copy Output and Open Output in Text Editor to Scrollable text output context menu.

* Context menu should be triggered on container

* fix naming
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Aug 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants