James Booth
James Booth
This is an easy add and would be great, will try and add sometime soon!
@grigorisg9gr thanks greg! Hmm when you said it failed to converge, what was the failure case? I'm wondering if instead this should be a case where an error is raised,...
@grigorisg9gr yeah I think that's what @patricksnape means, and I agree, that's the best design.
@patricksnape any reason to hold off on getting this in now?
@patricksnape proposal - implement a setting for `n_active_components` so users won't stumble into weird bugs trying to set this
I can check this over when I split up the packages
+1 once conflicts solved
@patricksnape maybe a quick test though?
@menpobot test this please
@patricksnape no, https://github.com/menpo/menpofit/issues/97 includes it's own copy of this code. Think we are better to revisit integrating this into Menpo for 0.8 once we have seen how the menpofit pretained...