Skip to content

docs(notebook): truncate model.variables output - #5713

Open
hwhsu1231 wants to merge 1 commit into
pybamm-team:mainfrom
hwhsu1231-fork:docs/truncate-model-variables-output
Open

docs(notebook): truncate model.variables output#5713
hwhsu1231 wants to merge 1 commit into
pybamm-team:mainfrom
hwhsu1231-fork:docs/truncate-model-variables-output

Conversation

@hwhsu1231

Copy link
Copy Markdown
Contributor

Description

Truncate the model.variables cell output in Tutorial 6 by replacing ~510 lines of expression-tree repr with a ... placeholder. The original output exceeded Crowdin's per-string 65535-byte limit, blocking .pot file uploads for the pybamm-docs-l10n project.

Fixes #5712

Type of change

Internal-only PRs — refactor, docs, CI, tests — can skip this.

Important checks:

Please confirm the following before marking the PR as ready for review:

  • No style issues: nox -s pre-commit
  • All tests pass: nox -s tests
  • The documentation builds: nox -s doctests
  • Code is commented for hard-to-understand areas
  • Tests added that prove fix is effective or that feature works

@hwhsu1231
hwhsu1231 requested a review from a team as a code owner August 10, 2026 18:13
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@hwhsu1231 hwhsu1231 changed the title 📝 docs(notebook): truncate model.variables output docs(notebook): truncate model.variables output Aug 10, 2026
Truncate the `model.variables` cell output in Tutorial 6 by replacing
~510 lines of expression-tree repr with a `...` placeholder. The original
output exceeded Crowdin's per-string 65535-byte limit, blocking `.pot`
file uploads for the `pybamm-docs-l10n` project.
@hwhsu1231
hwhsu1231 force-pushed the docs/truncate-model-variables-output branch from 48a89e0 to e06232f Compare August 10, 2026 19:08
@hwhsu1231

Copy link
Copy Markdown
Contributor Author

cc: @BradyPlanden

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.

Crowdin POT upload error caused by long output of model.variables in tutorial 6

1 participant