Skip to content

Conversation

@mmcky
Copy link
Contributor

@mmcky mmcky commented Nov 20, 2025

This PR tests the latest version of quantecon-book-theme (v0.11.0).

Changes:

  • Updated quantecon-book-theme from 0.10.1 to 0.11.0 in environment.yml

Release notes: https://github.com/QuantEcon/quantecon-book-theme/releases/tag/v0.11.0

This will trigger CI builds to verify compatibility with the new theme version.

@mmcky mmcky changed the title Test quantecon-book-theme v0.11.0 MAINT: upgrade quantecon-book-theme==v0.11.0 Nov 20, 2025
@github-actions
Copy link

📖 Netlify Preview Ready!

Preview URL: https://pr-720--sunny-cactus-210e3e.netlify.app (9db8649)

@mmcky
Copy link
Contributor Author

mmcky commented Nov 20, 2025

@jstac using linear_algebra as a test ground

https://pr-720--sunny-cactus-210e3e.netlify.app/linear_algebra#linear-independence

It looks like emphasis is used a lot instead of definition:

For example

In particular, a collection of vectors $A := {a_1, \ldots, a_k}$ in $\mathbb R ^n$ is said to be

  • linearly dependent if some strict subset of $A$ has the same span as $A$.
  • linearly independent if it is not linearly dependent.

--- Sonnet 4.5.

This is a definition. The text linearly dependent appears in a bulleted list where the concept is being formally defined for the first time. According to the QuantEcon style guide, this should use bold formatting instead of italic.

The correct formatting should be:

linearly dependent (bold for definition)
Similarly, on line 403, linearly independent should also be linearly independent since it's also being defined in that same context.

--- seems to do a pretty good job of identifying different cases.

@jstac
Copy link
Contributor

jstac commented Nov 20, 2025

It looks like emphasis is used a lot instead of definition:

Thanks @mmcky

Yes, we had no policy for this issue for a long time, so it's a complete mess.

(Hence the suggestion to unify --- but I can see why you rejected that idea.)

@mmcky
Copy link
Contributor Author

mmcky commented Nov 20, 2025

@mmcky mmcky merged commit 96d7d7f into main Nov 20, 2025
1 check passed
@mmcky mmcky deleted the test-theme-v0.11.0 branch November 20, 2025 08:46
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.

3 participants