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

[sf.cmath.assoc.laguerre,sf.cmath.assoc.legendre] Add reference to eq #7251

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

mattkretz
Copy link
Contributor

The associated Laguerre/Legendre functions build on the Laguerre/Legendre functions, which are defined in different equations. Point to them from the associated functions.

Also use the correct \ell as used in the formula.

The associated Laguerre/Legendre functions build on the
Laguerre/Legendre functions, which are defined in different equations.
Point to them from the associated functions.

Also use the correct \ell as used in the formula.
@mattkretz
Copy link
Contributor Author

Oh, there are more of such formula reuses. The other functions typically use a "See also" paragraph. My change makes the [sf.cmath] section less consistent.

Options:

  1. Undo the formula reference and add a "See also" paragraph instead.
  2. Add a "See also" paragraph and add formula references to all the other functions.
  3. Add formula references and remove "See also" paragraphs.

Personally, I find the formula reference to be clearer / more discoverable.

@jensmaurer
Copy link
Member

Since we normatively refer to ISO 80000-2, where all these symbols are introduced, I don't think we need to invest extra effort into cross-references. A "see also" seems good enough.

@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 16, 2024

Thanks!

@tkoeppe tkoeppe merged commit 9b6b757 into cplusplus:main Oct 16, 2024
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.

3 participants