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

Random fixes #34

Merged
merged 4 commits into from
Mar 30, 2024
Merged

Random fixes #34

merged 4 commits into from
Mar 30, 2024

Conversation

hmkemppainen
Copy link
Contributor

I noticed the section numbers in generated output did not match the text or input files. This is fixed by enabling pandoc attributes and making the preface unnumbered.

Also the internal links were all broken. Fixed by using header identifiers.

Note I couldn't get github to run the actions for me so I've only tested the pandoc changes locally with a version that may or may not match the version used on github.

There's still at least one broken link, chapter 12's matmul.k. I'll leave it to you to decide how to fix but I guess for pdf & epub users, simply using an absolute URL would work best.

I found a few more typos too.

Section numbers in output were off by one because the generated
document starte off with the preface as section number 1.

By enabling pandoc's attributes extension, we can make the Preface
section unnumbered.  Thus the introduction becomes the first chapter,
and the remaining chapters follow the numbering of the input files.
Pandoc does not care about input filenames, and thus the links to other
files in the book do not work.

What we can do is link to the autogenerated heading identifiers.  This
works even across multiple input files, because pandoc treats the input
as a single concatenated document.
@razetime razetime merged commit 8057ac0 into razetime:main Mar 30, 2024
1 of 2 checks passed
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.

2 participants