Performance optimization for builtin httpgd plot viewer#1635
Closed
TTTPOB wants to merge 10 commits intoREditorSupport:masterfrom
Closed
Performance optimization for builtin httpgd plot viewer#1635TTTPOB wants to merge 10 commits intoREditorSupport:masterfrom
httpgd plot viewer#1635TTTPOB wants to merge 10 commits intoREditorSupport:masterfrom
Conversation
added 5 commits
August 10, 2025 01:18
Since I didn't get the necessity of this condition?
Author
Also, when I run |
Author
|
Also, I am not sure about commit b3de22b. I don't know if this condition can be removed. |
httpgd plot viewerhttpgd plot viewer
added 4 commits
August 11, 2025 19:23
httpgd plot viewerhttpgd plot viewer
Author
|
I made some adjustment on the test suite issue, so it builds now(changes not in this pr). |
Author
|
I accidentally used my master branch to submit this pr so i have to close it. And I think maintainers seems have no spare effort in checking this PR so I will close it. If in the future any maintainer want to reopen it, can leave a message here. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related nx10/httpgd#113
Hi, this is my first PR to vscode-r, so please let me know if I’ve missed anything in the contribution process.
In the field of bioinformatics, R is often used for single-cell RNA-seq analysis, which typically involves visualizing datasets containing tens of thousands of points.
With the current httpgd SVG viewer in VS Code, rendering such dense plots can become very laggy and unresponsive.
On the other hand, if we disable
httpgdand fall back to the naïve PNG viewer, we lose useful features like zoom in/out and dynamic plot resizing.Motivation
Feature Proposal
Support PNG renderer in httpgd plot viewer
"png"as the renderer for the httpgd viewer, alongside existing formats.Plot history caching (planned, not yet implemented in this PR)
(plotIndex, width, height)to avoid unnecessary redraws when switching between plots of the same size and zoom level.Notes
Thanks for reviewing!