[jira] [Work logged] (MATH-1477) MillerUpdatingRegression: ArrayIndexOutOfBounds when calling regress with variablesToInclude parameter

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/MATH-1477?focusedWorklogId=225208&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-225208 ]

ASF GitHub Bot logged work on MATH-1477:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 09/Apr/19 18:37
            Start Date: 09/Apr/19 18:37
    Worklog Time Spent: 10m 
      Work Description: cheesinglee commented on pull request #104: MATH-1477: fix reordering check in MillerUpdatingRegression.regress
URL: https://github.com/apache/commons-math/pull/104
 
 
   Test and fix for MATH-1477
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

            Worklog Id:     (was: 225208)
            Time Spent: 10m
    Remaining Estimate: 0h

> MillerUpdatingRegression: ArrayIndexOutOfBounds when calling regress with variablesToInclude parameter
> ------------------------------------------------------------------------------------------------------
>
>                 Key: MATH-1477
>                 URL: https://issues.apache.org/jira/browse/MATH-1477
>             Project: Commons Math
>          Issue Type: Bug
>    Affects Versions: 3.6.1
>            Reporter: Chee Sing Lee
>            Priority: Major
>              Labels: easyfix, newbie, patch
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> When performing a regression with a subset of predictors, an ArrayIndexOutOfBounds exception will occur for certain subset selections. For example, this appears to happen consistently for regressions without a constant term when the predictor at index 0 is not selected.
> I do not understand at all algorithm used to reorder the predictors when a subset is used, but the fix appears to be a simple correction to the indexing range in the for-loop at line 1059 of MillerUpdatingRegression.java.
>  
> Patch with expanded unit test to follow shortly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)