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

Iris Flower Classification #84

Merged
merged 8 commits into from
Oct 5, 2024

Conversation

aviralgarg05
Copy link
Contributor

Iris Flower Classification using Machine Learning.
Solving Issue #79
GSSOC'24 Extended and Hacktoberfest

Copy link

github-actions bot commented Oct 4, 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! 😊

@Neilblaze
Copy link
Contributor

@aviralgarg05 LGTM! One suggestion that I'd provide is to add screenshots (preferably.gif's) with the README file. Once done, I'll approve the PR. Thanks for contributing! ✨

@aviralgarg05
Copy link
Contributor Author

There aren't as such Images or gifs, there are only graphs. Rest the accuracy is calculated on testing data(split from training dataset).
If you want I can put all graph images, confusion matrices etc.

@Neilblaze
Copy link
Contributor

I referred to the graph images, and to your point, sure you can add more images, but do keep those in a separate section in README.

@UppuluriKalyani
Copy link
Owner

UppuluriKalyani commented Oct 4, 2024

@aviralgarg05 ok you have added twitter sentiment analysis and iris flower classification right...and please add readme file with clear description and also results in that readme file for iris classification.

@aviralgarg05
Copy link
Contributor Author

Ok I will make all the changes

@aviralgarg05
Copy link
Contributor Author

All Changes have been done kindly review

@UppuluriKalyani UppuluriKalyani merged commit 624a43f into UppuluriKalyani:main Oct 5, 2024
1 check passed
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.

3 participants