search

UCD4IDS / ContinuousWavelets.jl

wide array of continuous wavelet transforms using Julia
https://dsweber2.github.io/ContinuousWavelets.jl/dev/
Other
21 stars 7 forks source link

Adjust syntaxes #28

Closed henry2004y closed 2 years ago

henry2004y commented 2 years ago

This PR is mostly about syntaxes, so all the functionalities should keep unchanged.

When scanning through the codes, I clearly sensed the remaining MATLAB-styles in many parts. Hopefully this PR can improve on the readability of the source codes.

In src/CWTConstruction.jl, I removed the broadcast for order within calculateProperties. All tests passed on my local laptop, but there is a chance that these changes are not covered by the current test suite.

dsweber2 commented 2 years ago

In src/CWTConstruction.jl, I removed the broadcast for order within calculateProperties. All tests passed on my local laptop, but there is a chance that these changes are not covered by the current test suite.

Not sure what you're referring to here, as order.( isn't present. Could you link the line?