Changes between Version 3 and Version 4 of Ticket #745, comment 29
- Timestamp:
- 07/06/17 11:02:27 (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #745, comment 29
v3 v4 8 8 * ~~Coefficient paths => Coefficients paths or just Coefficients ?~~ 9 9 * ~~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.~~ 11 11 12 12 I 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.