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

add another deprecated internal function for backwards compat #36794

Merged
merged 1 commit into from
Aug 28, 2020

Conversation

JeffBezanson
Copy link
Sponsor Member

ColorTypes uses this, so it's needed to allow packages to work on master.

@timholy
Copy link
Sponsor Member

timholy commented Jul 24, 2020

xref JuliaGraphics/ColorTypes.jl#209

@JeffBezanson
Copy link
Sponsor Member Author

Great! If that turns out to be the only user we can just close this.

@timholy timholy closed this Aug 1, 2020
@fredrikekre
Copy link
Member

Some (older) versions of Compat uses this as well, so maybe we can merge this after all? Failure e.g. here: https://travis-ci.org/github/JuliaLang/Pkg.jl/jobs/716196188#L203

@JeffBezanson JeffBezanson reopened this Aug 27, 2020
@JeffBezanson JeffBezanson added this to the 1.6 features milestone Aug 27, 2020
@JeffBezanson
Copy link
Sponsor Member Author

This is used by more packages, so reopening.

@fredrikekre
Copy link
Member

I ended up fixing it for Compat (JuliaLang/Compat.jl#717) but maybe there were more packages?

@KristofferC
Copy link
Sponsor Member

KristofferC commented Aug 27, 2020

It's also about packages that are stuck on older versions of dependencies. Can't do much about those.

@KristofferC KristofferC merged commit 4b42f14 into master Aug 28, 2020
@KristofferC KristofferC deleted the jb/p_u_bcompat branch August 28, 2020 06:49
oscardssmith added a commit to oscardssmith/julia that referenced this pull request Sep 14, 2020
author Oscar Smith <[email protected]> 1595400985 -0400
committer Oscar Smith <[email protected]> 1600122971 -0500

parent a0a68a5
author Oscar Smith <[email protected]> 1595400985 -0400
committer Oscar Smith <[email protected]> 1600122964 -0500

parent a0a68a5
author Oscar Smith <[email protected]> 1595400985 -0400
committer Oscar Smith <[email protected]> 1600122905 -0500

A faster version of exp for Float64

This is based on the Glibc algorithm which @chriselrod (Elrond on discourse) described the algorithm of for me. It appears to be about 2x faster than the current algorithm, and equally accurate over the range for which I have tried it. It also theoretically should be easier to vectorize as branches are only used for checking for over/underflow.

Update base/special/exp.jl

Co-authored-by: Jeff Bezanson <[email protected]>

Better subnormal numbers, constant usage, and more accurate

Break r into a hi and lo part to get extra accuracy.

Fix previous comit.

Error matches gexp

Switch to minimax polynomial from taylor polynomial.

equally fast version with a smaller table.

Uses a quartic which allows a smaller table. Performance is equal to slightly better, and accuracy is similar.

fully working?

fix markdown rendering (JuliaLang#37235)

add another deprecated internal function for backwards compat (JuliaLang#36794)

Remove unnecessary volatile on memcpy (JuliaLang#37221)

This is a local bitcast of different size through memory and doesn't need to be volatile.
This was introduced due to a typo in 8e4327c when the
argument order changed and the old tbaa parameter was passed in as isvolatile.

aysnchronous typos (JuliaLang#37264)

use printf/exit instead of `jl_error` for "too many threads" (JuliaLang#37223)

This is too early for `jl_error` to work.

fix expm1 for Float32 (calling wrong libm function)

actually fix expm1

re-add exp10 docs

slightly extend upper range for Float32 arguments

re-add exp doctest, remove redundant exp10 doctest

maybe now

placing it after

use round instead of magic

re-add magic with better explanation

remove the typo

Fix all numbers taking the slow path with range checking.

Oops

Fix 32 bit build.

Use `:ℯ` instead of `float64 ℯ`

no functional change but less hacky

Update base/special/exp.jl

Co-authored-by: jmert <[email protected]>

Update base/special/exp.jl

Co-authored-by: jmert <[email protected]>

Update base/special/exp.jl

Co-authored-by: jmert <[email protected]>
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.

None yet

4 participants