James Booth

Results 70 comments of 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...