ransac algorithm has been modified to support mulitiple dispatch for (y, X) and use interal ols methods instead of GLM package.
The existing test case passes so there shouldn't be any new introduced errors. One thing though was that initially ols encountered singularity exception, and I had to wrap the calls to it in a try-catch block. I don't think ols is supposed to throw an exception there though. Maybe if ols uses pinv then it can be handled better?
ransac
algorithm has been modified to support mulitiple dispatch for (y, X) and use interalols
methods instead ofGLM
package.The existing test case passes so there shouldn't be any new introduced errors. One thing though was that initially
ols
encountered singularity exception, and I had to wrap the calls to it in a try-catch block. I don't thinkols
is supposed to throw an exception there though. Maybe ifols
usespinv
then it can be handled better?