Modified Linear Regression to work on OLS, fixes #8847 #11311
Conversation
for more information, see https://pre-commit.ci
smruthi-sumanth
changed the title
There was a problem hiding this comment.
Good stuff! Apart from the changes that I've requested in my other comments, could you also rewrite the explanation at the top of the file to reflect your new implementation? In particular, make sure your new explanation does the following:
- Briefly explain what linear regression is
- Explain the OLS regression formula,
$(\mathbf{X}^{\top} \mathbf{X})^{-1} \mathbf{X} \mathbf{y}$ - Note possible issues with your implementation (i.e. inefficiency, numerical instability, etc.)
- Cite a source that readers can refer to for further info (Wikipedia or similar is fine)
I wrote a similar explanation for weighted regression in machine_learning/local_weighted_learning/local_weighted_learning.py a while back. If you'd like, you're welcome to use that explanation as a reference (though your explanation doesn't need to be nearly as long).
| :param data_x : contains our dataset | ||
| :param data_y : contains the output (result vector) | ||
| def run_linear_regression_ols(data_x, data_y): | ||
| """Implement Linear regression using OLS over the dataset |
There was a problem hiding this comment.
| """Implement Linear regression using OLS over the dataset | |
| """Implement OLS linear regression over a given dataset |
Slight rewording for clarity
| :return : feature for line of best fit (Feature vector) | ||
| """ |
There was a problem hiding this comment.
The OLS regression function needs doctests—make sure you verify the outputs of your tests with a calculator that can do linear regression (e.g., Wolfram Alpha)
tianyizheng02
added
awaiting changes
Co-authored-by: Tianyi Zheng <[email protected]>
Co-authored-by: Tianyi Zheng <[email protected]>
|
Closing tests_are_failing PRs to prepare for Hacktoberfest 2024 |
Describe your change:
Checklist: