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

Lazy snippet generation #65

Closed
wants to merge 3 commits into from
Closed

Lazy snippet generation #65

wants to merge 3 commits into from

Conversation

Casteran
Copy link
Member

@Zimmi48 On this branch I put systematically .. coq:: none blocks around snippet-free regions of Epsilon0/T1.v and Epsilon0/Paths.v The size of T1.tex was reduced by a factor of 70, and Paths.tex by a factor of 336 !
Do you think this can help to reduce documentation generation time too?

@Casteran Casteran requested a review from Zimmi48 August 23, 2021 11:57
@Zimmi48
Copy link
Member

Zimmi48 commented Aug 23, 2021

Sure, it should help at least a little by avoiding from requesting intermediate goals that are not printed, writing to and reading from disk, and speed up the snippet extraction script. That being said, it's not an important enough speed up to show in the CI numbers. I think that most of the time spent by Alectryon is the time that Coq takes to check proof scripts themselves (whether they are printed or not doesn't change this).

@cpitclaudel
Copy link
Contributor

This looks good. I this we could also achieve this without annotations, by adding a transform that removes all irrelevant code before it gets fed to LaTeX (this is similar to what's being done here, but without annotations.)

@Zimmi48
Copy link
Member

Zimmi48 commented Aug 31, 2021

I think this has been superseded by #71 and could be closed, right?

@cpitclaudel
Copy link
Contributor

cpitclaudel commented Aug 31, 2021

Yep (at least on the movies/ side)

@Casteran
Copy link
Member Author

Casteran commented Aug 31, 2021 via email

@Zimmi48 Zimmi48 closed this Aug 31, 2021
@Casteran Casteran deleted the lazy-snippet-generation branch October 11, 2021 12:04
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.

None yet

3 participants