8000 Add basic docstrings to some functions by JSorngard · Pull Request #337 · rust-lang/libm · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Apr 28, 2025. It is now read-only.

Add basic docstrings to some functions #337

Merged
merged 12 commits into from
Oct 28, 2024
8000 Merged

Add basic docstrings to some functions #337

merged 12 commits into from
Oct 28, 2024

Conversation

JSorngard
Copy link
Contributor
@JSorngard JSorngard commented Oct 28, 2024

This PR adds basic docstrings for some of the functions in the crate.

Related to #336.

@JSorngard JSorngard marked this pull request as ready for review October 28, 2024 12:02
Copy link
Contributor
@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

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

Thanks! One request but mostly lgtm.

…o the docstrings of all Bessel functions.
Copy link
Contributor
@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

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

Thank you!

@tgross35 tgross35 merged commit c3ab752 into rust-lang:master Oct 28, 2024
14 checks passed
@github-actions github-actions bot mentioned this pull request Oct 28, 2024
tgross35 pushed a commit that referenced this pull request Apr 18, 2025
* Add docstring to Bessel functions
* Add docstrings to logarithm functions
* Add docstrings to pow functions
* Specify argument bit-size of the Bessel functions
* Specify argument bit-size for pow functions
* Specify argument bit-size for logarithms
* Add docstrings to sin, cos, sincos and sinh functions
* Add docstrings to sqrt
* Add docstrings to tan and tanh functions
* Add an inline link to https://en.wikipedia.org/wiki/Bessel_function to the docstrings of all Bessel functions.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0