Package name conflict

[timholy]

I fear this package may have the same problem that the Iterators package has been experiencing:

julia> module Mine
       using FFTW

       struct Foo end

       FFTW.fft(::Foo) = 1

       end
WARNING: both FFTW and Base export "FFTW"; uses of it in module Mine must be qualified
ERROR: UndefVarError: FFTW not defined

julia> versioninfo()
Julia Version 0.7.0-DEV.616
Commit d49469d* (2017-06-16 15:45 UTC)
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-3.9.1 (ORCJIT, broadwell)
Environment:
  JULIAHOME = /home/tim/src/julia
  JULIAFUNCDIR = /home/tim/juliafunc
  JULIA_CPU_CORES = 2

[timholy]

Or, following the advice of the docs:

julia> module Mine
       importall FFTW

       struct Foo end

       FFTW.fft(::Foo) = 1

       end
Mine

julia> x = Mine.Foo()
Mine.Foo()

julia> fft(x)
ERROR: fft has been moved to the package FFTW.jl.
Run `Pkg.add("FFTW")` to install FFTW then run `using FFTW` to load it.
Stacktrace:
 [1] #fft#6(::Array{Any,1}, ::Function, ::Mine.Foo, ::Vararg{Mine.Foo,N} where N) at ./deprecated.jl:1448
 [2] fft(::Mine.Foo, ::Vararg{Mine.Foo,N} where N) at ./deprecated.jl:1448

It doesn't help to get rid of the module name in front of the fft specialization.

[ararslan]

I can't verify at the moment, but the workaround should be

using FFTW
importall FFTW

[timholy]

Nope. Get an error on usage like the one in my 2nd post.

[ararslan]

Ah, I see, we posted at the same time. :) This package doesn't export anything while FFTW is defined in Base (though I have an unmerged PR here that should make them extend the Base functions when they're deprecated), so you'll have to do using and/or importall outside of the module definition as well.

[timholy]

julia> using FFTW

julia> importall FFTW

julia> module Mine
       using FFTW
       importall FFTW

       struct Foo end

       fft(::Foo) = 1

       end
Mine

julia> x = Mine.Foo()
Mine.Foo()

julia> fft(x)
ERROR: fft has been moved to the package FFTW.jl.
Run `Pkg.add("FFTW")` to install FFTW then run `using FFTW` to load it.
Stacktrace:
 [1] #fft#6(::Array{Any,1}, ::Function, ::Mine.Foo, ::Vararg{Mine.Foo,N} where N) at ./deprecated.jl:1448
 [2] fft(::Mine.Foo, ::Vararg{Mine.Foo,N} where N) at ./deprecated.jl:1448

[ararslan]

Can you try it after doing Pkg.checkout("FFTW"); Pkg.checkout("FFTW", "aa/testing")?

[timholy]

Same error message.

julia> Pkg.status("FFTW")
 - FFTW                          0.0.1+             aa/testing

[ararslan]

Maybe

using AbstractFFTs, FFTW
importall AbstractFFTs, FFTW

?

[timholy]

Same error.

[ararslan]

Well, I have no idea what's going on here. :/

[ararslan]

This is really concerning. I'm not sure what to do about it. Requesting backup: @stevengj and @tkelman.

[musm]

hmm, I too can't get the build to actually follow through:

julia> BinDeps.debug("FFTW")
INFO: Reading build script...
The package declares 2 dependencies.
INFO: Updating WinRPM package list
INFO: Downloading https://cache.julialang.org/http://download.opensuse.org/repositories/windows:/mingw:/win32/openSUSE_Leap_42.2/repodata/repomd.xml
INFO: Downloading https://cache.julialang.org/http://download.opensuse.org/repositories/windows:/mingw:/win64/openSUSE_Leap_42.2/repodata/repomd.xml
 - Library "libfftw3"
    - Providers:
      - BinDeps.AptGet package libfftw3-double3 (can't provide)
      - BinDeps.Pacman package fftw (can't provide)
      - BinDeps.Zypper package libfftw3 (can't provide)
      - BinDeps.Yum package fftw (can't provide)
      - BinDeps.BSDPkg package fftw3 (can't provide)
      - WinRPM.RPM package fftw (can't provide)
      - Simple Build Process
 - Library "libfftw3f"
    - Providers:
      - BinDeps.AptGet package libfftw3-single3 (can't provide)
      - BinDeps.Pacman package fftw (can't provide)
      - BinDeps.Zypper package libfftw3 (can't provide)
      - BinDeps.Yum package fftw (can't provide)
      - BinDeps.BSDPkg package fftw3 (can't provide)
      - WinRPM.RPM package fftw (can't provide)
      - Simple Build Process

Are you sure the WinRPM search is set correctly, e.g.

julia> WinRPM.search("fftw")
WinRPM Package Set:
  1. fftw3-debug (mingw64) - Debug information for package mingw64-fftw3
  2. fftw3-debugsource (mingw64) - Debug sources for package mingw64-fftw3
  3. fftw3-devel (mingw64) - Include Files and Libraries mandatory for Development
  4. libfftw3-3 (mingw64) - Discrete Fourier Transform (DFT) C Subroutine Library

[appleparan]

Seems no problem with fresh install of Julia ver. 0.6.0. Here is test code.

using Base.Test
using FFTW

# simple 1D fft testt
a = rand(8) + im*rand(8)
@test norm(FFTW.ifft(FFTW.fft(a)) - a) < 1e-8

# simple 2D fft test
a = rand(8, 8) + im * rand(8, 8)
@test norm(FFTW.ifft(FFTW.fft(a)) - a) < 1e-8

# 1D FFTW C2R test
a = rand(8) + im * rand(8)
@test norm(FFTW.fft(FFTW.ifft(a)) - a) < 1e-8

# 1D Convolution Test
a = conv(vec([1 2 3]),vec([1 2 3]))
b = FFTW.ifft(FFTW.fft(vec([1 2 3 0 0])) .* FFTW.fft(vec([1 2 3 0 0])))
c = FFTW.ifft(FFTW.fft(vec([1 2 3])) .* FFTW.fft(vec([1 2 3])))
@test norm(a - b) < 1e-8

# 2D FFT Test
a = rand(8, 8) + im * rand(8, 8)
@test norm(FFTW.fft(FFTW.ifft(a)) - a) < 1e-8

[musm]

@appleparan The bug is on master, where the bindings have been removed. On 0.6 it will simply use the shipped bindings so it's not a problem