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

feat/docs/fix ✨ Added momentum based gradient descent to increase the accuracy #42

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

ayushi2626
Copy link
Contributor

Used momentum-based gradient descent and it increased the accuracy. I have attached the screenshots below:

Accuracy_ss

SampleData_ss

Copy link

github-actions bot commented Oct 1, 2024

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@UppuluriKalyani
Copy link
Owner

@ayushi2626 hey even though the accuracy is 90 above but the prediction isn't well..it's wrongly classifying please check once

@ayushi2626
Copy link
Contributor Author

Hey are you talking about the predictions using the images one before saving the model and one after saving the model?

@UppuluriKalyani
Copy link
Owner

UppuluriKalyani commented Oct 2, 2024

@ayushi2626 I think both🥲, if it is classifying 2 or 3 images it's ok but it actually predicting wrong for all.. except 2 or 3...so please train data again and save the model and make predictions.

@ayushi2626
Copy link
Contributor Author

Sure, let me check it once

@ayushi2626
Copy link
Contributor Author

Hey I just pushed into the same issue here. So sorry I am new to open-sourcing so please do let me know if there is anything more to do done. Thank you so much!

@UppuluriKalyani
Copy link
Owner

@ayushi2626 it's ok don't worry, it's predicting good afternoon saving the model but not on training data...ok you did it good.

@UppuluriKalyani UppuluriKalyani merged commit d10d52c into UppuluriKalyani:main Oct 2, 2024
1 check passed
@ayushi2626
Copy link
Contributor Author

Thank you so much!

@ayushi2626 ayushi2626 deleted the fix-issue-36 branch October 2, 2024 10:20
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.

2 participants