Skip to content

fix: exporting a large figure hangs#442

Merged
camdecoster merged 4 commits intoplotly:masterfrom
EliasTalcott:419-exporting-a-large-figure-hangs
Apr 22, 2026
Merged

fix: exporting a large figure hangs#442
camdecoster merged 4 commits intoplotly:masterfrom
EliasTalcott:419-exporting-a-large-figure-hangs

Conversation

@EliasTalcott
Copy link
Copy Markdown
Contributor

Choreographer uses remote debugging pipes to execute Javascript functions in Chrome. These have a receive buffer size of 100MB (reference). Previously, if a spec exceeded this size, we would try to send it anyways and the image export would never complete. This makes it so we chunk large specs to keep each send within the buffer size.

Closes #419

@EliasTalcott EliasTalcott force-pushed the 419-exporting-a-large-figure-hangs branch from 6031959 to 24108b2 Compare April 10, 2026 18:18
Choreographer uses remote debugging pipes to execute Javascript
functions in Chrome. These have a receive buffer size of 100MB.
Previously, if a spec exceeded this size, we would try to send it
anyways and the image export would never complete. This makes it
so we chunk large specs to keep each send within the buffer size.

chrome ref: https://chromium.googlesource.com/chromium/src/+/refs/heads/main/content/browser/devtools/devtools_pipe_handler.cc#44
issue: plotly#419
@EliasTalcott EliasTalcott force-pushed the 419-exporting-a-large-figure-hangs branch from 24108b2 to 7f1a39c Compare April 10, 2026 20:14
@camdecoster camdecoster self-assigned this Apr 22, 2026
Copy link
Copy Markdown
Contributor

@camdecoster camdecoster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@camdecoster camdecoster merged commit 8c86e4c into plotly:master Apr 22, 2026
@EliasTalcott EliasTalcott deleted the 419-exporting-a-large-figure-hangs branch April 22, 2026 19:29
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.

write_image hangs with large amount of data

3 participants