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

Remove the FFTW bindings from Base #21956

Merged
merged 1 commit into from
Jun 16, 2017
Merged

Remove the FFTW bindings from Base #21956

merged 1 commit into from
Jun 16, 2017

Conversation

ararslan
Copy link
Member

@ararslan ararslan commented May 18, 2017

Ready to merge!

This PR completely removes FFTW from Base. The bindings will instead be available in the FFTW.jl package in the JuliaMath organization. I'm still working on the package, but in the meantime I figured I'd put this up here to give folks a preview of what's to come.

Fixes #18389

@ararslan ararslan added deprecation This change introduces or involves a deprecation maths Mathematical functions labels May 18, 2017
@ararslan ararslan force-pushed the aa/farewell-fftw branch 3 times, most recently from 22a71f2 to 65bb63f Compare May 23, 2017 18:54
@ararslan
Copy link
Member Author

@jlbuild !nuke

Pkg.clone("https://github.com/JuliaMath/FFTW.jl.git")
Pkg.checkout("FFTW", "master")
Pkg.build("FFTW")
Pkg.test("FFTW")

@jlbuild
Copy link

jlbuild commented May 24, 2017

Status of 65bb63f builds:

Builder Name Nuke Build Download Code Output
linux32 COMPLETE COMPLETE Download COMPLETE
linux64 COMPLETE COMPLETE Download ERRORED
linuxaarch64 COMPLETE COMPLETE Download ERRORED
linuxarmv7l COMPLETE COMPLETE Download COMPLETE
linuxppc64le COMPLETE COMPLETE Download COMPLETE
osx64 COMPLETE COMPLETE Download PENDING
win32 COMPLETE ERRORED N/A N/A
win64 COMPLETE ERRORED N/A N/A

@ararslan
Copy link
Member Author

This is getting really close. This PR, AbstractFFTs, and FFTW are ready to go, I just need to finish up the DSP package.

@ararslan ararslan changed the title WIP: Remove the FFTW bindings from Base Remove the FFTW bindings from Base Jun 15, 2017
@ararslan
Copy link
Member Author

The packages that inherited these functions are ready now, so this is good to go!

@ararslan ararslan requested a review from tkelman June 15, 2017 21:27
@ararslan
Copy link
Member Author

Four hours in the CI queue after a rebase, but now it's good to go.

@StefanKarpinski
Copy link
Member

do it

@ararslan ararslan merged commit 00b9ea2 into master Jun 16, 2017
@ararslan ararslan deleted the aa/farewell-fftw branch June 16, 2017 02:11
@ararslan
Copy link
Member Author

i did it

@StefanKarpinski
Copy link
Member

Probably should have tried this before we merged this, but I'm seeing this:

julia> fft(rand(10))
ERROR: UndefVarError: pkg not defined
Stacktrace:
 [1] #fft#6(::Array{Any,1}, ::Function, ::Array{Float64,1}, ::Vararg{Array{Float64,1},N} where N) at ./deprecated.jl:1448
 [2] fft(::Array{Float64,1}, ::Vararg{Array{Float64,1},N} where N) at ./deprecated.jl:1448

@JeffBezanson
Copy link
Member

Ah, string interpolation strikes again :trollface:

@ararslan
Copy link
Member Author

Ah crap. Will fix.

@ararslan
Copy link
Member Author

Fix in #22391

ararslan added a commit that referenced this pull request Jun 16, 2017
quinnj pushed a commit that referenced this pull request Jun 20, 2017
@martinholters
Copy link
Member

martinholters commented Sep 21, 2017

Something is still not quite as it should here (or am I missing something?), but the problem might be with DSP.jl?

   _       _ _(_)_     |  A fresh approach to technical computing
  (_)     | (_) (_)    |  Documentation: https://docs.julialang.org
   _ _   _| |_  __ _   |  Type "?help" for help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 0.7.0-DEV.1776 (2017-09-14 03:47 UTC)
 _/ |\__'_|_|_|\__'_|  |  Commit deceb3be66* (7 days old master)
|__/                   |  x86_64-linux-gnu

julia> Pkg.status("DSP")
 - DSP                           0.3.2

julia> using DSP

julia> xcorr([1, 2], [3, 4])
ERROR: Base.DSP.xcorr has been moved to the package DSP.jl.
Run `Pkg.add("DSP")` to install it, restart Julia,
and then run `using DSP` to load it.
Stacktrace:
 [1] error(::Function, ::String, ::String, ::String, ::String, ::String, ::String, ::String, ::String, ::String) at ./error.jl:44
 [2] #xcorr#6(::Array{Any,1}, ::Function, ::Array{Int64,1}, ::Vararg{Array{Int64,1},N} where N) at ./deprecated.jl:126
 [3] xcorr(::Array{Int64,1}, ::Vararg{Array{Int64,1},N} where N) at ./deprecated.jl:126

OTOH:

julia> using DSP: xcorr

julia> xcorr([1, 2], [3, 4])
3-element Array{Int64,1}:
  4
 11
  6

Edit: Fixed in DSP.jl.

@StefanKarpinski StefanKarpinski added the excision Removal of code from Base or the repository label Sep 21, 2017
vchuravy pushed a commit to JuliaPackaging/LazyArtifacts.jl that referenced this pull request Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecation This change introduces or involves a deprecation excision Removal of code from Base or the repository maths Mathematical functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants