[PR #904] [MERGED] Fixing inject_crossrefs.lua #999

New Issue

GiteaMirror · 2026-03-22T15:57:26-05:00

GiteaMirror commented

2026-03-22 15:57:26 -05:00

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/904
Author: @hzeljko
Created: 7/24/2025
Status: ✅ Merged
Merged: 7/24/2025
Merged by: @profvjreddi

Base: dev ← Head: 899-fixing-inject_crossrefslua

📝 Commits (2)

b20e725 Fixing inject_crossrefs.lua
cfa6d95 Merge remote-tracking branch 'origin/dev' into 899-fixing-inject_crossrefslua

📊 Changes

11 files changed (+141 additions, -48 deletions)

View changed files

📝 _extensions/custom-numbered-blocks/style/foldbox.tex (+1 -0)
📝 _quarto.yml (+3 -2)
📝 contents/core/data_engineering/data_engineering.qmd (+11 -3)
📝 contents/core/dl_primer/dl_primer.qmd (+13 -2)
📝 contents/core/dnn_architectures/dnn_architectures.qmd (+9 -1)
📝 contents/core/introduction/introduction.qmd (+10 -3)
📝 contents/core/ml_systems/ml_systems.qmd (+9 -1)
📝 contents/core/workflow/workflow.qmd (+9 -1)
📝 lua/inject_crossrefs.lua (+57 -34)
📝 style.scss (+4 -0)
📝 tex/header-includes.tex (+15 -1)

📄 Description

Commit description: Inject \Needspace before each section with a callout in the Lua filter (PDF only) to avoid orphaned headings and red tcolorbox errors, and add CSS break-inside rules for HTML to keep margin-callout boxes intact. Predefined caption for figures and tables

Before submitting your Pull Request, please review and complete the items on this checklist.

The text has been proofread for grammar and spelling errors.
All images, figures, and tables render properly without any glitches.
All images have a source or they are properly linked to external sites.
The chapter's formatting is consistent with the rest of the book.
The chapter has been locally built and tested using Quarto.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/904 **Author:** [@hzeljko](https://github.com/hzeljko) **Created:** 7/24/2025 **Status:** ✅ Merged **Merged:** 7/24/2025 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `899-fixing-inject_crossrefslua` --- ### 📝 Commits (2) - [`b20e725`](https://github.com/harvard-edge/cs249r_book/commit/b20e725fd5b9f9fb75fb346587ecae94d16a44da) Fixing inject_crossrefs.lua - [`cfa6d95`](https://github.com/harvard-edge/cs249r_book/commit/cfa6d95955d98f99f1da12836b2f698c1f165fd1) Merge remote-tracking branch 'origin/dev' into 899-fixing-inject_crossrefslua ### 📊 Changes **11 files changed** (+141 additions, -48 deletions) <details> <summary>View changed files</summary> 📝 `_extensions/custom-numbered-blocks/style/foldbox.tex` (+1 -0) 📝 `_quarto.yml` (+3 -2) 📝 `contents/core/data_engineering/data_engineering.qmd` (+11 -3) 📝 `contents/core/dl_primer/dl_primer.qmd` (+13 -2) 📝 `contents/core/dnn_architectures/dnn_architectures.qmd` (+9 -1) 📝 `contents/core/introduction/introduction.qmd` (+10 -3) 📝 `contents/core/ml_systems/ml_systems.qmd` (+9 -1) 📝 `contents/core/workflow/workflow.qmd` (+9 -1) 📝 `lua/inject_crossrefs.lua` (+57 -34) 📝 `style.scss` (+4 -0) 📝 `tex/header-includes.tex` (+15 -1) </details> ### 📄 Description Commit description: Inject \Needspace before each section with a callout in the Lua filter (PDF only) to avoid orphaned headings and red tcolorbox errors, and add CSS break-inside rules for HTML to keep margin-callout boxes intact. Predefined caption for figures and tables Before submitting your Pull Request, please review and complete the items on this checklist. - [x] The text has been proofread for grammar and spelling errors. - [x] All images, figures, and tables render properly without any glitches. - [x] All images have a source or they are properly linked to external sites. - [x] The chapter's formatting is consistent with the rest of the book. - [x] The chapter has been locally built and tested using Quarto. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-03-22 15:57:26 -05:00

GiteaMirror closed this issue