DiffEqOperators.jl icon indicating copy to clipboard operation
DiffEqOperators.jl copied to clipboard

Method definition *(Number, SciMLBase.AbstractDiffEqLinearOperator{T}) overwritten in module DiffEqOperators

Open oashour opened this issue 5 years ago • 2 comments

I'm using DifferentialEquations.jl and DiffEqOperators.jl in my package (Julia 1.5.2). Recently this error started popping up when precompiling my package.

WARNING: Method definition *(Number, SciMLBase.AbstractDiffEqLinearOperator{T}) where {T} in module SciMLBase at /Users/ashour/.julia/packages/SciMLBase/HlKbQ/src/operators/common_defaults.jl:15 overwritten in module DiffEqOperators at /Users/ashour/.julia/packages/DiffEqOperators/viN6A/src/composite_operators.jl:20.
  ** incremental compilation may be fatally broken for this module **

I am not sure if I'm doing something wrong in my package or what's up, I'm relatively new to Julia. The packages installed are:

Project NonlinearSchrodinger v0.0.5
Status `~/.julia/dev/NLSS/Project.toml`
  [a93c6f00] DataFrames v0.22.5
  [9fdde737] DiffEqOperators v4.21.0             <====
  [0c46a032] DifferentialEquations v6.16.0   <====
  [b305315f] Elliptic v1.0.1
  [7a1cc6ca] FFTW v1.3.1
  [916415d5] Images v0.23.3
  [b964fa9f] LaTeXStrings v1.2.0
  [6fafb56a] Memoization v0.1.5
  [91a5bcdd] Plots v1.10.3
  [5d786b92] TerminalLoggers v0.1.3
  [56ddb016] Logging

I can't reproduce this at the REPL just by activating the environment and running using DifferentialEquations, DiffEqOperators

Any thoughts?

oashour avatar Feb 07 '21 19:02 oashour

It's at least harmless. It looks like it was introduced in:

https://github.com/SciML/SciMLBase.jl/commit/87f08cf819c865e55868a8dfcef7856b4a01de6c

@YingboMa https://github.com/SciML/DiffEqOperators.jl/pull/330 it looks like this library engaged in a bit of type-piracy which caused it. Do you think we could drop the Number definition for now? Anything come to mind for it?

ChrisRackauckas avatar Feb 08 '21 00:02 ChrisRackauckas

Thanks as always Chris, I'll just ignore it for now. Wish I was knowledgeable enough to help fix this :)

oashour avatar Feb 08 '21 02:02 oashour