Possible regression using quadgk function and Unitful limits about quadgk.jl HOT 9 CLOSED

I just found a related bug report on the Unitful.jl package repo.

from quadgk.jl.

The funny thing is that it is failing on:

  [2] one(#unused#::Type{Quantity{Float64, 𝐓, Unitful.FreeUnits{(s,), 𝐓, nothing}}}  )
    @ Base .\number.jl:334

which is this fallback method of one(T). That seems like Unitful's fault — it should really be dispatching to a one method in Unitful.

from quadgk.jl.

I just tried it with the latest versions (QuadGK 2.6.0 and Unitful v1.12.2) in Julia 1.8.2, and it works fine for me:

julia> using Unitful, QuadGK

julia> f(t) = 1.0u"J"
f (generic function with 1 method)

julia> quadgk(f, 0.0u"s", 10.0u"s")
(10.0 J s, 0.0 J s)

So something is weird on your system?

from quadgk.jl.

Hmm, interesting. I can sometimes reproduce your error. If I enter the code above line-by-line, it works. But if I paste it in all at once, then I get the error. This seems like a bug in Julia, TBH.

from quadgk.jl.

Aha, I just noticed that the difference between your code and mine: I was doing using Unitful, QuadGK whereas you were doing using QuadGK, Unitful.

It seems like this is a bug in Julia, since it dispatch shouldn't depend on the order of the imports: JuliaLang/julia#48295

from quadgk.jl.

Fascinating. I can reproduce the import-order issue that you detected in both a prior install of Julia v1.7.3 and a fresh install of Julia Version 1.9.0-beta2, Commit 7daffeecb8 (2022-12-29 07:45 UTC) with only those two packages added.

from quadgk.jl.

Should be fixed now.

from quadgk.jl.

Awesome. Thanks!

from quadgk.jl.

Maybe could add a test?

from quadgk.jl.