|
It looks like I accidentally checked the "not intended for inclusion" radio button. This obviously is intended for inclusion and you have my full permission to commit it.
This is an interesting addition to commons-math.
I think it would be better to put the clustering package under stat. Perhaps later we could add an EM implementation too. Could you rework your patch to include the ASF license header and complete javadoc including algorithms references, parameters and return values descriptions ? thanks Here are the updated files with the requested changes. Thanks for taking a look!
fixed in subversion repository as of r770979
I have changed a few things:
Could you please check if everything is fine and either reopen the issue if something is wrong or open a new one if you want to add something ? thanks for the report and for the patch Thanks Luc. It all looks good to me from what I've seen. I appreciate the
quick response. -Ben closing resolved issue for 2.0 release
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
I tested it out by using it for image segmentation and it worked well (reducing # of colors in image by clustering on pixel values - for an example see http://www.leet.it/home/lale/joomla/component/option,com_wrapper/Itemid,50/