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

Show data fields only to avoid "not rendered due to size" #160

Open
alreadydone opened this issue Nov 12, 2023 · 1 comment
Open

Show data fields only to avoid "not rendered due to size" #160

alreadydone opened this issue Nov 12, 2023 · 1 comment

Comments

@alreadydone
Copy link

Some pretty modest definitions, like nonZeroDivisors, are not shown in the docs, and the message "One or more equations did not get rendered due to their size." is shown instead. Even though the proofs take four lines in this case, the only data field carrier := { x | ∀ z, z * x = 0 → z = 0 } is quite simple and people shouldn't need to open a new page to view the definition in the source.

I guess maybe doc-gen sees the term-mode proofs generated by the tactics, which can be quite long, so it decides not to show the definition in the docs; if it sees ._proof_1 etc. maybe it will show them, but not showing proofs at all seems to be a better option.

@hargoniX
Copy link
Collaborator

doc-gen has two approaches to rendering equations.

One is to use the simplification equations that you get for free. However these are not available on all definitions. In this situation it will instead decide to render basically defName = defBody. And I would not really like to process the body myself in an effort to figure out where potentially omittable things are hidden. If there is some option that disables rendering Prop stuff in terms we can definitely enable that and see how it goes though.

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

No branches or pull requests

2 participants