Skip to content

Conversation

@jcheng5
Copy link
Member

@jcheng5 jcheng5 commented May 22, 2023

No description provided.

jcheng5 added 6 commits May 21, 2023 20:53
This implements server_no_context_takeover and client_no_context_takeover
support, but in a pretty naive way for now that gives correct results but doesn’t provide any
memory efficiency advantage, which is the whole point of this feature.
@jcheng5 jcheng5 force-pushed the joe/feature/permessage-deflate branch from e950415 to 058f75b Compare May 22, 2023 04:48
@jcheng5 jcheng5 force-pushed the joe/feature/permessage-deflate branch from 058f75b to 1762ce7 Compare May 22, 2023 04:50
@zeehio
Copy link

zeehio commented Oct 25, 2023

On shiny websites with plotly plots with a large number of points, generating the plot is the first bottleneck. Once that is cached (bindCache...) the transfer of the plotly object from server to client becomes the bottleneck.

Having websocket per-message compression would reduce the cost of the transfer at the expense of compressing/decompressing each message.

Ideally, having some websocket-based caching system would be awesome, so the web browser does not download the same plot more than once. I would imagine httpuv sending a hash of the response first, and then, if the browser does not have anything matching that hash, the full response (maybe with compression as implemented here).

Thanks for all this work anyway... It's easy to suggest features, but it is much harder to implement them!

@anthonynorth
Copy link

Looking for permessage-deflate support in shiny led me here. Are there plans to merge this into main?

@TBlackmore
Copy link

Also came here looking to speed up shiny messages. Can I upvote this feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants