Free cookie consent management tool by TermsFeed Policy Generator

Changes between Version 3 and Version 4 of Ticket #745, comment 29


Ignore:
Timestamp:
07/06/17 11:02:27 (7 years ago)
Author:
gkronber
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #745, comment 29

    v3 v4  
    88* ~~Coefficient paths => Coefficients paths or just Coefficients ?~~
    99* ~~Line 165 trainRsq & testRsq should be renamed to NMSE (train & test).~~
    10 * The method name 'CreateElasticNetLinearRegressionSolution' is misleading, because actually no solution is created but rather the coefficients & intercept are calculated.
     10* ~~The method name 'CreateElasticNetLinearRegressionSolution' is misleading, because actually no solution is created but rather the coefficients & intercept are calculated.~~
    1111
    1212I have reviewed the implementation from line 217 until the end only briefly, but I tested it thoroughly using the feature selection benchmark problems and it works pretty well! During testing i got some further ideas for improvements listed below that are not necessary for trunk integration, but would ease working with elastic net regression and improve the source code.