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

smart-phone-analysis #697

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Conversation

BenakDeepak
Copy link
Contributor

Here's a concise update based on the format you've specified:


Related Issues or Bug

  • Lack of additional predictive features may limit model accuracy.
  • Dataset might need further cleaning to reflect realistic data distributions.

Fixes: #688

Proposed Changes

  • Added columns like brand, model, and storage capacity as predictors for improved accuracy.
  • Filled NaN values in the Rating column and adjusted the distribution to ensure realistic values.
  • Optimized RandomForestRegressor with GridSearchCV for hyperparameter tuning.
  • Compared RandomForestRegressor with GradientBoostingRegressor and XGBoost for model selection.
  • Created log transformation on Reviews due to high range and added interaction terms between Rating and Reviews.

Additional Info

  • Conducted exploratory data analysis (EDA) to understand data patterns and significant predictors.
  • Added Mean Absolute Error (MAE) as an evaluation metric alongside RMSE for more comprehensive error assessment.

Screenshots

Original Updated
Original screenshot Updated screenshot

Copy link

👋 Thank you for opening this pull request! We appreciate your contribution to improving this project. Your PR is under review, and we'll get back to you shortly.
Don't forget to mention the issue you solved!.

To help move the process along, please tag @UppuluriKalyani, @Neilblaze, and @SaiNivedh26 for a faster review!

@UppuluriKalyani UppuluriKalyani merged commit 574e2e6 into UppuluriKalyani:main Oct 31, 2024
3 checks passed
Copy link

🎉🎉 Thank you for your contribution! Your PR #697 has been merged! 🎉🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request Smart-Phone prediction
2 participants