Skip to content

Commit

Permalink
Decrease max transmittable output from 10MB to 5MB (#108)
Browse files Browse the repository at this point in the history
Due to user reports of accidentally outputting large outputs
still crashing their frontend
  • Loading branch information
akshayka authored Sep 14, 2023
1 parent 6fcbb57 commit d0f7a54
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions marimo/_messaging/streams.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,14 @@
#
# 2. The frontend chokes when we send outputs that are too big, i.e.
# it freezes and sometimes even crashes. That can lead to lost work.
# It appears this is the bottleneck right now, compared to 1.
#
# Usually users only output gigantic things accidentally, so refusing
# to show large outputs should in most cases not bother the user too much.
# In any case, it's better than breaking the frontend/kernel.
#
# Output not shown if larger than OUTPUT_MAX_BYTES=10MB
OUTPUT_MAX_BYTES = 10_000_000
# Output not shown if larger than OUTPUT_MAX_BYTES=5MB
OUTPUT_MAX_BYTES = 5_000_000

# Standard stream truncated if larger than STD_STREAM_MAX_BYTES=1MB
STD_STREAM_MAX_BYTES = 1_000_000
Expand Down

1 comment on commit d0f7a54

@vercel
Copy link

@vercel vercel bot commented on d0f7a54 Sep 14, 2023

Choose a reason for hiding this comment

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

Successfully deployed to the following URLs:

marimo-storybook – ./frontend

marimo-storybook-git-main-marimo.vercel.app
marimo-storybook.vercel.app
marimo-storybook-marimo.vercel.app

Please sign in to comment.