advy99 / advy99/cpp_ml

LinearRegression should be a shorcut to polynomial_regression(degree = 1)

Open
#8 4 comments 0 reactions 1 assignee Claimed by @frey0-0 View on GitHub
enhancement good first issue hacktoberfest
Dominant language
C++
Stars
2
Forks
3
PR merge metrics
No merged PRs in 30d

Description

With #7 there is no sense of mantaining both linear_regression and polynomial_regression, as linear_regression is a particular case of polynomial_regression with degree = 1

All the code of linear_regression should be deleted, and because Linear Regression is widely used, create a shorcut to polynomial_regression(degree = 1) with the name of linear_regression (maybe a typedef is enough??)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.