Hi, On 01/30/2014 09:06 AM, Manuel Grizonnet wrote: > BTW I am asking myself if we should really have tagged this class as > deprecated in fact...This class was marked as deprecated when the new > MachineLearningModel framework was added. This new framework wrapped > mainly OpenCV classifiers but we have also wrapped LibSVM machine > learning model in the new framework to avoid to add OpenCV as mandatory > to perform classification in the lib. That the purpose of the > LibSVMMachineLearningModel The point is that the > LibSVMMachineLearningModel use the SVMSampleListModelEstimator > internally to . So if we keep libSVM implementation, there is no reason > to mark SVMSampleListModelEstimator as deprecated > ...
Good point. Actually the original idea was to mark the old SVM related classes deprecated to drive users to the new interface if they write new code based on the SVM functionnality. This current thread is a proof that it is actually doing the job. Using the old classes for the implementation of LibSVMMachineLearningModel is only an implementation detail. My opinion is that LibSVMMachineLearningModel shall ideally be reimplemented to use the libsvm API more directly. We can probably save some layers of data copy. When this is done, we may remove of the old classes. Julien -- -- Check the OTB FAQ at http://www.orfeo-toolbox.org/FAQ.html You received this message because you are subscribed to the Google Groups "otb-users" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/otb-users?hl=en --- You received this message because you are subscribed to the Google Groups "otb-users" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
