ricedsp / rwt

The Rice Wavelet Toolbox (RWT) is a collection of Matlab M-files and C MEX-files for 1D and 2D wavelet and filter bank design, analysis, and processing. The toolbox provides tools for denoising and interfaces directly with our Matlab code for wavelet domain hidden Markov models and wavelet regularized deconvolution.
http://dsp.rice.edu
Other
135 stars 47 forks source link

mdwt,midwt enhancements #13

Open mborgerding opened 7 years ago

mborgerding commented 7 years ago

mdwt and midwt now handle

Tests were added for new modes.

This branch closes issue #12

kalliste commented 7 years ago

This is great.

Rice has asked me to get a contributor agreement from everyone for this project - can we get you to sign one?

https://mega.nz/#!sEJ2AIaa!hMBS2sXCgrH2UWq7oX8_VhY-1O3UHZW1cM-r3Ep8N8U

mborgerding commented 7 years ago

I sent the CLA to your email a week ago. What's the status?

kalliste commented 7 years ago

Your contribution looks great. Before merging it I'll still want to personally verify:

Also will want to deal with:

Currently no one is being paid to work on wavelet toolbox so I am guessing that this will have to wait until I clear out my queue of paying work.

mborgerding commented 6 years ago

bump

kalliste commented 5 years ago

I ought to have stated clearly what would be required to get this merged before asking someone to go to the trouble of signing a contributor agreement. Sorry about that - that's on me to be a better maintainer.

I won't merge this change for mdwt and midwt without matching changes for rdwt and irdwt and also the code style should be kept consistent across the whole package.

After watching a video of Howard Hinnant looking at the assembly output of his C++ date and time code vs C I have concluded that I am ok with the language switch. Also the hypothetical Rice EE undergraduate students can get some tough love and learn to read C++ template code.

That just leaves regression testing and getting someone to convince me that this is the correct way to handle complex values. If someone cares enough to handle the rdwt and irdwt implementations and consistent code formatting then I will plan to match their effort by doing the regression testing and reaching out to a professor or someone about complex values.