search

munch2024 / munch

2 stars 11 forks source link

Fixes #84 #85

Closed tadakane closed 8 months ago

tadakane commented 8 months ago

Fixes #84

Start of complex code process for issue #84. The current state of the code is hard to understand and complex with long statements of matrix operations. There is no clear indication as to what the code snippet is calculating and is unnecessarily convoluted.

Post-Refactoring: After the ChatGPT conversation, the code is a lot simpler. It uses one function to calculate the linear regression coefficient vector instead of multiple matrix operations being applied over and over. This makes it easier to understand and also look into if any reader has more questions about the function. It utilizes built-in functions from the numpy library which can help with efficiency. Additionally, I asked ChatGPT to add documentation for what equation the linear regression coefficient vector is derived from to further the clarity of the code snippet and the calculation being done.

ChatGPT link of complex code conversation: https://chat.openai.com/share/e8a57986-62ce-4c5e-8cd3-6f0cd21dfdd9

rparker2003 commented 8 months ago

I reviewed the changes and ChatGPT very clearly explained the code you were requesting assistance understanding. Great example using SQL!