Skip to content

docs: standardize LaTeX math notation in model-structure.md - #280

Merged
Alomir merged 5 commits into
PecanProject:masterfrom
man080107:docs/fix-math-notation-consistency
Mar 5, 2026
Merged

docs: standardize LaTeX math notation in model-structure.md#280
Alomir merged 5 commits into
PecanProject:masterfrom
man080107:docs/fix-math-notation-consistency

Conversation

@man080107

Copy link
Copy Markdown
Contributor

Summary

Standardized LaTeX notation across SIPNET documentation by resolving inconsistent use of \mathit, \mathrm, \text, and bare symbols, especially for CH₄ flux subscripts and water-dependency functions.

Changes

  • Used \text{} for descriptive subscripts (e.g., litter, soil, water, vol).
  • Standardized methane notation to \text{CH}_4 throughout.
  • Replaced \mathrm{} with \operatorname{} for functions (e.g., clip).
  • Fixed inconsistent notation in flux terms (F), rate constants (K), and dependency functions (D).
  • Updated headings and inline math for consistency.

Motivation

Ensures consistent, semantically correct LaTeX formatting and improves documentation readability.

Testing

Built documentation locally and verified equations render correctly. Changes are documentation-only and do not affect model outputs.

Related issues

Fixes #277

Checklist

  • Related issue referenced
  • Documentation updated
  • No functional code changes

Resolve mixed use of \mathit, \mathrm, \text, and bare (no command)
for similar symbols throughout the document. The inconsistency was
most visible in CH4-related flux subscripts and water-dependency
function labels.

Notation convention applied:
- Descriptive subscript text ('litter', 'soil', 'vol', 'water', etc.):
  use \text{} uniformly.
- Chemical formulas in math (CH4): use \text{CH}_4 throughout,
  with the numeric subscript placed outside \text{} so it renders
  as a proper mathematical subscript.
- Math operator / function names (clip): use \operatorname{} instead
  of \mathrm{}, which is the semantically correct LaTeX command for
  named operators and prevents incorrect italic spacing.

Specific changes:
- F^C_{CH4,litter} / F^C_{CH4,soil}: \mathit -> \text{CH}_4\text{,...}
- Section heading 'Methane Production (C -> CH4)': bare -> \text{CH}_4
- Equation (eq:ch4): \mathit{CH_4,j} -> \text{CH}_4\text{,}j for
  both the flux symbol F^C and the rate parameter K.
- D_\mathrm{water, CH_4}: \mathrm -> \text{} and split to
  D_{\text{water,CH}_4} for consistency with all other D_{water,...}.
- Inline prose F^C_{CH_4}, K_{\mathit{CH_4,}j}: standardized.
- F^N_\mathrm{vol} (two occurrences): \mathrm -> \text{}.
- D_{water,N_vol} subscript (heading + inline): N_vol -> N_\text{vol}.
- D_{\mathrm{water},N_{vol}} inline: \mathrm -> \text{}.
- D_{water,CH_4} heading + equation: bare CH_4 -> \text{CH}_4.
- \\$ in prose: bare -> \text{CH}_4.
- clip() function: \mathrm{clip} -> \operatorname{clip} (both sites).

@dlebauer dlebauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thanks! Could you please also check that parameters.md has consistent typesetting?

@man080107

Copy link
Copy Markdown
Contributor Author

Hi @dlebauer , I've pushed the latest changes! Following your advice, I've standardized all the LaTeX math notation across both
model-structure.md
and
parameters.md

. Methane (CH₄) and Nitrous Oxide (N₂O) formulas are now properly formatted using upright roman text (\text{}), and I've replaced all instances of \textit{}, \textrm{}, and \mathfrak{} with standard \text{}. Let me know if everything looks good!"

@man080107
man080107 requested a review from dlebauer February 28, 2026 18:20
Comment thread docs/parameters.md Outdated
@man080107

Copy link
Copy Markdown
Contributor Author

Thanks @Alomir for pointing that out. I’ve removed both rows for the initial CH₄ and N₂O soil concentrations as suggested. Please let me know if anything else needs to be adjusted.

@dlebauer
dlebauer dismissed their stale review March 3, 2026 18:31

requested changes have been addressed

@dlebauer
dlebauer requested a review from Alomir March 3, 2026 18:31

@Alomir Alomir left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks!

@Alomir
Alomir merged commit 4baf19a into PecanProject:master Mar 5, 2026
11 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.

Standardize LaTeX notation in model-structure.md and parameters.md

3 participants