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

Very large console outputs can make Chrome tab unresponsive #1380

Closed
akshayka opened this issue May 15, 2024 · 0 comments · Fixed by #1424
Closed

Very large console outputs can make Chrome tab unresponsive #1380

akshayka opened this issue May 15, 2024 · 0 comments · Fixed by #1424
Assignees
Labels
bug Something isn't working

Comments

@akshayka
Copy link
Contributor

Describe the bug

If you create a cell that prints a LOT of console outputs, the frontend becomes unresponsive (hard to scroll, things don't render, buttons don't work) and Chrome prompts you to kill it.

It's never a good idea to print this much stuff, but sometimes it happens accidentally -- in this case better to show the last N lines rather than become unresponsive.

Additionally, this affected the entire Home server and all open notebooks -- all open tabs became unresponsive -- even though only one of the notebooks had the extremely large console output.

Environment

{
  "marimo": "0.5.2",
  "OS": "Linux",
  "OS Version": "6.5.0-27-generic",
  "Processor": "x86_64",
  "Python Version": "3.10.12",
  "Binaries": {
    "Browser": "123.0.6312.122",
    "Node": "v18.13.0"
  },
  "Requirements": {
    "click": "8.1.3",
    "importlib-resources": "6.1.0",
    "jedi": "0.19.0",
    "markdown": "3.4.4",
    "pymdown-extensions": "10.3",
    "pygments": "2.16.1",
    "tomlkit": "0.12.0",
    "uvicorn": "0.24.0.post1",
    "starlette": "0.27.0",
    "websocket": "missing",
    "typing-extensions": "4.8.0",
    "black": "23.3.0"
  }
}

Code to reproduce

for i in range(1000000):
    print(i)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant