JuliaHealth / NeuroAnalyzer.jl

Julia toolbox for analyzing neurophysiological data
https://neuroanalyzer.org
BSD 2-Clause "Simplified" License
5 stars 1 forks source link

[COMPAT MISMATCH] Not Compatible with ComplexityMeasures.jl? #1

Open TheCedarPrince opened 3 hours ago

TheCedarPrince commented 3 hours ago

Hey @AdamWysokinski,

I've been doing some work with @datseris using ComplexityMeasures.jl and was curious if you could help me debug why using NeuroAnalyzer.jl reverts the latest version of ComplexityMeasures.jl (3.6.6) down to version 2.8. I did some investigation myself and concluded that the compat mismatch was because NeuroAnalyzer permits version 0.9 and 0.10 of Wavelets.jl whereas ComplexityMeasures.jl only permits 0.10.

I am not entirely sure how to reconcile the mismatch between the two packages other than forking ComplexityMeasures.jl and adding a lower bound to Wavelets.jl to include version 0.9. Any thoughts Adam and/or George?

Cheers!

~ tcp :deciduous_tree:

AdamWysokinski commented 2 hours ago

Hi Jacob, There is a "man in the middle" that causes the problem - WaveletsExt.jl requires Wavelets 0.9.3, thus I had to added 0.9 requirements here. I'll submit a PR to them with the fix. However, it might be necessary to fork the WaveletsExt.jl since my last PR to this repo has not been reviewed for the last two years. Seems like the repo is dead.. ATB, Adam

TheCedarPrince commented 2 hours ago

Oh -- yea, we may want to fork that, update it with your fixes, and move that into JuliaDSP. It looks unmaintained to me as well @AdamWysokinski.