Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removal of Classification Metrics #37

Conversation

AnujSaha0111
Copy link

@AnujSaha0111 AnujSaha0111 commented Oct 4, 2024

Closes #35

updating Regression metrics
In project which involves Stock Price Prediction which is a regression tasks, the objective is to predict continuous values, making these metrics irrelevant and potentially misleading when evaluating model performance.

Since metrics like Mean Absolute Error (MAE), Mean Squared Error (MSE), Root Mean Squared Error (RMSE), and Mean Absolute Percentage Error (MAPE) are already implemented and are appropriate for this regression task, I suggest removing the classification metrics. This will help maintain clarity and focus on relevant performance indicators.

updating Regression metrics
In project which involves Stock Price Prediction which is a regression tasks, the objective is to predict continuous values, making these metrics irrelevant and potentially misleading when evaluating model performance.

Since metrics like Mean Absolute Error (MAE), Mean Squared Error (MSE), Root Mean Squared Error (RMSE), and Mean Absolute Percentage Error (MAPE) are already implemented and are appropriate for this regression task, I suggest removing the classification metrics. This will help maintain clarity and focus on relevant performance indicators.
@AnujSaha0111
Copy link
Author

AnujSaha0111 commented Oct 5, 2024

Following Issue #35 is solved

@jvedsaqib
Copy link
Collaborator

@AnujSaha0111 please resolve the conflict

@AnujSaha0111
Copy link
Author

facing issues in resolving the conflicts

Copy link
Collaborator

@jvedsaqib jvedsaqib left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please synchronize the project first, then proceed to make the necessary changes. Once the code has been updated, kindly commit the changes and submit a pull request.

@jvedsaqib jvedsaqib added the level2 25 Points 🥈(GSSoC) label Oct 13, 2024
@Mayureshd-18 Mayureshd-18 self-requested a review November 5, 2024 04:57
Copy link
Collaborator

@Mayureshd-18 Mayureshd-18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AnujSaha0111 Its been a long time since you have created this PR. Please update, resolve and make the requested changes as soon as possible.

Thanks and regards

@Mayureshd-18 Mayureshd-18 self-requested a review November 9, 2024 13:53
Copy link
Collaborator

@Mayureshd-18 Mayureshd-18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing this PR for long period of inactivity. Feel free to reopen!

Thanks,
Mayuresh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level2 25 Points 🥈(GSSoC)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Removal of Classification Metrics from Regression Model Evaluation
3 participants