Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

M-estimators #67

Open
Marco-Congedo opened this issue Feb 26, 2020 · 11 comments
Open

M-estimators #67

Marco-Congedo opened this issue Feb 26, 2020 · 11 comments

Comments

@Marco-Congedo
Copy link

Hi,

referring to the list in issue #8 ,
i implemented the classical Tyler's M-estimator (1987) and the shrinked version proposed by Zhang and Wiesel (2016), with both the Ledoit & Wolf-type of shrinkage and the one advocated by the authors based on random matrix theory. The good news is that Zhang and Wiesel's estimator is pretty efficient, with a computational complexity comparable to the classical Tyler's M-estimator.

Are you interested in putting them in this package?

REFERENCES
David E. Tyler (1987)
A Distribution-Free M-Estimator of Multivariate Scatter
The Annals of Statistics, 15(1), 234-251.

Teng Zhang, Ami Wiesel (2016)
Automatic diagonal loading for Tyler's robust covariance estimator
IEEE Statistical Signal Processing Workshop (SSP), 1-5.

@tlienart
Copy link
Collaborator

Definitely, thanks!

@Marco-Congedo
Copy link
Author

Nice, actually the two algorithms are stand-alone, they only requires LinearAlgebra.
Let me commit the .jl to start with, so that you can check and tell me how you see the support for those estimators should be done. Ok?

@mateuszbaran
Copy link
Owner

Thanks! I can look at your code and give some tips.

@Marco-Congedo
Copy link
Author

OK, let me clean up a bit the code and i will commit.

@Marco-Congedo
Copy link
Author

I committed. You will also find code for testing and benchmarking the algorithms as they stand. If you have any question do not hesitate to ask.

@mateuszbaran
Copy link
Owner

Thanks but where did you commit it?

@Marco-Congedo
Copy link
Author

Sorry, now it should be OK.

@mateuszbaran
Copy link
Owner

Sorry but I can't find it, could you paste a link here?

@mateuszbaran
Copy link
Owner

OK, I see it now 👍 .

@Marco-Congedo
Copy link
Author

Hello, any news on the addition of the Tyler's type M-estimator?

@mateuszbaran
Copy link
Owner

Sorry for this delay but it looks like it still requires a considerable amount of work. I don't currently need covariance estimation for anything so I keep prioritizing other tasks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants