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

MVU does not work #183

Closed
rcurtin opened this Issue Dec 29, 2014 · 2 comments

Comments

Projects
None yet
1 participant
@rcurtin
Member

rcurtin commented Dec 29, 2014

Reported by rcurtin on 8 Jun 42063788 04:40 UTC
I can't get it to successfully unroll even simple swiss roll problems. I suspect this is related to #188, or, it may be that my implementation of the MVU objective function is incorrect.

@rcurtin rcurtin self-assigned this Dec 29, 2014

@rcurtin rcurtin added this to the mlpack 1.0.7 milestone Dec 29, 2014

@rcurtin rcurtin closed this Dec 29, 2014

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin Dec 30, 2014

Member

Commented by rcurtin on 13 Mar 42171200 20:32 UTC
I've made progress, and I think it may converge for very simple cases, but I'm not certain yet. Pushing this back to a later release.

Member

rcurtin commented Dec 30, 2014

Commented by rcurtin on 13 Mar 42171200 20:32 UTC
I've made progress, and I think it may converge for very simple cases, but I'm not certain yet. Pushing this back to a later release.

@rcurtin

This comment has been minimized.

Show comment
Hide comment
@rcurtin

rcurtin Dec 30, 2014

Member

Commented by rcurtin on 18 Jul 43759720 19:36 UTC
I have been unable to resolve this so I am closing it wontfix. The code is in trunk still (although I may remove it in the future), but comments are provided to note that it does not work, and it is not compiled into mlpack.

Member

rcurtin commented Dec 30, 2014

Commented by rcurtin on 18 Jul 43759720 19:36 UTC
I have been unable to resolve this so I am closing it wontfix. The code is in trunk still (although I may remove it in the future), but comments are provided to note that it does not work, and it is not compiled into mlpack.

@sbrodehl sbrodehl referenced this issue Jan 12, 2018

Merged

cmake project #1190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment