[mlpack-svn] [MLPACK] #178: No parameter validation in constructors (or elsewhere)

MLPACK Trac trac at coffeetalk-1.cc.gatech.edu
Tue Dec 20 23:14:10 EST 2011


#178: No parameter validation in constructors (or elsewhere)
---------------------+------------------------------------------------------
 Reporter:  rcurtin  |        Owner:                         
     Type:  defect   |       Status:  new                    
 Priority:  minor    |    Milestone:  MLPACK 1.1.0           
Component:  MLPACK   |     Keywords:  validation constructors
 Blocking:  177      |   Blocked By:                         
---------------------+------------------------------------------------------
 In accordance with a recent discussion, parameter validation should only
 be done in a `Validate()` method, and therefore should not be done in a
 constructor or anything like that, for speed reasons.  If a user inputs a
 bad model, and the model fails, that is their fault.  Their first step for
 debugging should be trying the `Validate()` function and seeing if it
 outputs any errors.

 We may even see a few speed gains from doing this.

-- 
Ticket URL: <http://trac.research.cc.gatech.edu/fastlab/ticket/178>
MLPACK <www.fast-lab.org>
MLPACK is an intuitive, fast, and scalable C++ machine learning library developed by the FASTLAB at Georgia Tech under Dr. Alex Gray.


More information about the mlpack-svn mailing list