ConstrainedClassifier should receive node
Opened this issue · 1 comments
kordjamshidi commented
After the latest changes for using nodes instead of the whole datamodel, it seems one thing has left without being updated, that is, ConstrainedClassifier. I think @bhargav was completing these related changes so assigned to him.
bhargav commented
Christos's changes removed the DataModel instance that was passed into ConstrainedClassifier. In the current implementation, ConstrainedClassifier uses either pathToHead
or the onClassifier
instance to get its training/testing data. There is no need to explicitly provide a node.