search

bnels / EMD.jl

⛔️ DEPRECATED Empirical Mode Decomposition Package
MIT License
3 stars 4 forks source link

errors on trying to use the package with v1.1.0 #1

Open mantzaris opened 5 years ago

mantzaris commented 5 years ago

I'm running Julia v1.1.0 on Debian9, and added the repo, and upon trying to 'use' the module have produced errors;

ulia> using EMD
[ Info: Precompiling EMD [3c2051a6-7368-5338-aae3-7c17c25c36d9]
ERROR: LoadError: syntax: space required before "?" operator
Stacktrace:
 [1] include at ./boot.jl:326 [inlined]
 [2] include_relative(::Module, ::String) at ./loading.jl:1038
 [3] include(::Module, ::String) at ./sysimg.jl:29
 [4] top-level scope at none:2
 [5] eval at ./boot.jl:328 [inlined]
 [6] eval(::Expr) at ./client.jl:404
 [7] top-level scope at ./none:3
in expression starting at /home/xxx/.julia/packages/EMD/XUPSg/src/EMD.jl:46
ERROR: Failed to precompile EMD [3c2051a6-7368-5338-aae3-7c17c25c36d9] to /home/xxx/.julia/compiled/v1.1/EMD/FUPew.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1197
 [3] _require(::Base.PkgId) at ./loading.jl:960
 [4] require(::Base.PkgId) at ./loading.jl:858
 [5] require(::Module, ::Symbol) at ./loading.jl:853

a similarly natured issue arises with Splines

victorminden commented 5 years ago

Thanks for the ping -- this package hasn't been touched / updated in several years and it looks like syntax has gotten stricter since then.

@mantzaris, if you want to try to fix this locally you could download the files and modify the lines in these files (and in splines as appropriate) to add a space before each "?". I would bet that there are other issues that arise as well after this, however.

@bnels, maybe we should try the quick fix, and slap a deprecated sticker on this? I would make changes but then I have to get code approved before submission...

bnels commented 5 years ago

Apologies for the delay. Was travelling.

@victorminden I updated the package description and README to indicate that the package is no longer being maintained.

@gmaher If you are interested in updating the repository, you're welcome to

@mantzaris unfortunately I don't have the bandwidth to resurrect the repository right now. However, you're welcome to make a pull request if you've fixed it up, or fork the repository for your own use.

gmaher commented 5 years ago

@bnels and @victorminden thanks for responding to this. Unfortunately I do not have the bandwidth to make this a priority either.

mantzaris commented 5 years ago

Apologies for the delay. Was travelling.

@victorminden I updated the package description and README to indicate that the package is no longer being maintained.

@gmaher If you are interested in updating the repository, you're welcome to

@mantzaris unfortunately I don't have the bandwidth to resurrect the repository right now. However, you're welcome to make a pull request if you've fixed it up, or fork the repository for your own use.

I will look into it, over the weekend

victorminden commented 5 years ago

Thanks, Brad!

On Thu, May 30, 2019, 6:27 PM gmaher notifications@github.com wrote:

@bnels https://github.com/bnels and @victorminden https://github.com/victorminden thanks for responding to this. Unfortunately I do not have the bandwidth to make this a priority either.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bnels/EMD.jl/issues/1?email_source=notifications&email_token=AAN32E2YK4ZXDJYOMX7AXO3PYBIGVA5CNFSM4HOQECCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWTV54A#issuecomment-497508080, or mute the thread https://github.com/notifications/unsubscribe-auth/AAN32EYF3QH46KMIBUCQNNTPYBIGVANCNFSM4HOQECCA .