Skip to content

Conversation

@jorenham
Copy link
Owner

@jorenham jorenham commented Nov 9, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 9, 2025 22:47
@jorenham jorenham added the documentation Improvements or additions to documentation label Nov 9, 2025
@jorenham jorenham added this to the 0.3.5 milestone Nov 9, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request refactors mathematical notation throughout the documentation to use a more concise and consistent LaTeX/KaTeX syntax. The changes simplify macro definitions and update their usage across the codebase.

Key changes:

  • Updated KaTeX macro definitions to use more compact syntax (e.g., \rpow x m instead of \rpow{x}{m})
  • Changed throwOnError from true to false in the KaTeX configuration
  • Standardized hypergeometric function notation and added new macros (\abs, \I)

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
katex.html Updated macro definitions to use String.raw syntax, removed deprecated Kelvin function derivative macros (\berp, \beip, \kerp, \keip), added \abs and \I macros, and set throwOnError: false
src/lib.rs Updated mathematical notation in documentation tables to use the new compact macro syntax and corrected table column alignment
src/xsf/specfun.rs Simplified hypergeometric function notation in documentation, updated to use compact \hyp macro syntax
src/xsf/hyp2f1.rs Updated mathematical formulas to use compact macro syntax and improved consistency of hypergeometric notation
src/scipy_special/hypergeometric.rs Refactored hypergeometric function documentation to use the new notation style, updated "See also" sections for consistency
src/scipy_special/convex_analysis.rs Changed list notation from \ldots to \dotsc for comma-separated sequences

@jorenham jorenham requested a review from Copilot November 10, 2025 02:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

@jorenham jorenham merged commit 0d678d3 into master Nov 10, 2025
13 checks passed
@jorenham jorenham deleted the katex-tweaks branch November 10, 2025 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant