search

ahwillia / Einsum.jl

Einstein summation notation in Julia
MIT License
151 stars 17 forks source link

Julia 0.6 deprecation warning messages #26

Closed RaulDurand closed 5 years ago

RaulDurand commented 6 years ago

When using Einsum in Julia 0.6 we get the message:

WARNING: Array{T}(::Type{T}, m::Int, n::Int, o::Int) is deprecated, use Array{T}(m, n, o) instead.
phipsgabler commented 6 years ago

As far as I can see, there is only one place where Array is used, and that has been updated some time ago to remove this warning.

Which version are you using? Can you try to narrow down the problem?

ahwillia commented 6 years ago

They are fixed on master but we should tag a new version so that users don't need to run Pkg.checkout("Einsum"). Usually this is done with:

julia> PkgDev.tag("Einsum")
julia> PkgDev.publish()

But I'm running into an error, https://github.com/JuliaLang/PkgDev.jl/issues/101

mcabbott commented 5 years ago

Note that in #29 I added Compat, so that the undef syntax should work on both 1.0 and 0.6. But this error looks like it might be from a version for julia before 0.6.

phipsgabler commented 5 years ago

Seems to work on the current 0.6 build, and should be fixed in the next published release.