closes 7690, adds: @app.cell(expanded_output=True)#9141
closes 7690, adds: @app.cell(expanded_output=True)#9141csp256 wants to merge 2 commits intomarimo-team:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
recheck |
ce64b7a to
da143a9
Compare
for more information, see https://pre-commit.ci
|
Hey @csp256, in #7690 we decided against per-cell configuration. We were converging toward I think this needs a bit more discussion in the issue before proceeding for PR. There were unresolved questions of how a global expand outputs configuration would interact with possible configuration for text outputs. It would be good to study what other projects do as well, and form an opinion of what works and what doesn't |
|
I guess I will continue the discussion on the issue. While I agree broad defaults are a good idea, I think that the per-cell configurable behavior has some benefits. |
|
@akshayka Also, you should probably update CONTRIBUTING.md with the restriction that all public API changes need to be approved before PR. It currently just says:
I would not have considered this a large contribution. |
|
Thanks, we have updated the contributing guide to expand upon contributions #9153. |
📝 Summary
Closes #7690
This introduces the ability to annotate a cell with
@app.cell(expand_output=True)to cause that cell's output to be automatically expanded by default when the user loads the notebook.This PR does not provide a way to cause this behavior from the web UI.
📋 Pre-Review Checklist
✅ Merge Checklist
I have read the CLA Document and I hereby sign the CLA