-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Even MORE trig doc movement... #17871
Conversation
($fa){T<:Number}(y::T) = ($fainv)(one(T) / y) | ||
@doc """ | ||
$($name)(x) | ||
Compute the $($fn) of `x`, where the output is in radians. """ ($fa){T<:Number}(y::T) = ($fainv)(one(T) / y) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you run make docs
on this? These are inverses. Is the output in radians for the hyperbolic ones?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just by comparing the old doc strings to the new it looks like you forgot to put the word "inverse" before the function name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what happens when you write PRs on a delayed train
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Santa Barbara, the only place west of the Mississippi where Amtrak is the more convenient option?
edit: nope, guessed wrong
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a car and the San Diego airport is a pain to get to from the university. Free WiFi + comfy chair = so may doc PRs
and I fixed a failing doctest in the manual, so doctests pass.