-
Notifications
You must be signed in to change notification settings - Fork 861
draft: proof of concept for ipc kernels #7640
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
for more information, see https://pre-commit.ci
| # Set MARIMO_MANAGE_SCRIPT_METADATA for sandbox | ||
| env["MARIMO_MANAGE_SCRIPT_METADATA"] = "true" | ||
| else: | ||
| # We use threads in run mode to minimize memory consumption; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed we still need to use threads for run mode
| # for packaging.version; not sure what the lower bound is. | ||
| "packaging", | ||
| "msgspec>=0.20.0", | ||
| "pyzmq>=27.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pyzmq should probably be optional, only for marimo edit --sandbox FOLDER
|
Closing out for #7702 |
📝 Summary
Proof of concept regarding the feasibility of IPC controlled kernels. I expect to essentially rewrite this
Didn't want to go too deep with Lazy kernels also likely changing this code.
Considerations on revisiting: