search

JuliaDSP / Wavelets.jl

A Julia package for fast discrete wavelet transforms and utilities
Other
190 stars 29 forks source link

wplotim is destructive #71

Closed Dejan-Ilic closed 3 years ago

Dejan-Ilic commented 3 years ago

wplotim modifies the argument x it receives. I think either the function should be renamed to wplotim! or on line 83 of Plot.jl xts = x should be changed to xts = copy(x) (which seems the best option to me). Thanks for your time :)

JeffFessler commented 3 years ago

nice catch, thanks. a workaround while waiting for a new version is wplotim(copy(x))

gummif commented 3 years ago

Good catch, a fix has been merged.