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

Fix atahn to atanh #17977

Merged
merged 2 commits into from
Aug 12, 2016
Merged

Fix atahn to atanh #17977

merged 2 commits into from
Aug 12, 2016

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Aug 11, 2016

No description provided.

@kshyatt kshyatt added domain:maths Mathematical functions kind:bugfix This change fixes an existing bug labels Aug 11, 2016
@tkelman tkelman added needs tests Unit tests are required for this change backport pending 0.5 labels Aug 11, 2016
@tkelman
Copy link
Contributor

tkelman commented Aug 11, 2016

Thanks, but a @kshyatt PR that doesn't have tests? My world is shaken.

@kshyatt
Copy link
Contributor Author

kshyatt commented Aug 11, 2016

My doc prs don't have tests...

I'll see if I can trigger the bug on master. We do have tests for atanh so I'm confused that they didn't explode.

@tkelman
Copy link
Contributor

tkelman commented Aug 11, 2016

Judging by the packages that failed, it's probably triggered by defining a custom AbstractArray type that goes to the fallback - the existing tests for atanh are probably hitting more specialized methods?

@tkelman tkelman merged commit 5838a12 into master Aug 12, 2016
@tkelman tkelman deleted the ksh/atahn branch August 12, 2016 07:40
tkelman pushed a commit that referenced this pull request Aug 12, 2016
* Fix atahn to atanh

* More tests

(cherry picked from commit 5838a12)
@tkelman tkelman removed needs tests Unit tests are required for this change backport pending 0.5 labels Aug 12, 2016
mfasi pushed a commit to mfasi/julia that referenced this pull request Sep 5, 2016
* Fix atahn to atanh

* More tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:maths Mathematical functions kind:bugfix This change fixes an existing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants