-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Add Pluto and other Dwarf planets #88
Comments
Hey @ritwik12 , I'm interested in contributing to this issue, so before I start working on it, would you mind sparing your time explaining what the issue is about and pointing me to some resources to get started. |
Yeah. so just like we have 8 planets right now for the classification. Our model is not trained for Pluto and Dwarf planets. If we try to give an image of Pluto to our model. It will classify it as wrong. |
Need to add Training images for Pluto, then add test images. Train the model based on those images. |
Hello @ritwik12 , I'm interested in solving this issue, and I cloned this repository and I collected some pluto images and added them to both training and test data and added "pluto" to retrained_labels.txt and label_image.py and now how to start training ??is this the correct way ?? |
@SSahas Please follow the readme.md. I think you also don't need to update To train model you need to run |
hey @ritwik12 , I trained the model for classifying Pluto planet and got the final test accuracy : 87% , I also run the run.py and the webapp is classifying Pluto images correctly but the retrained_graph.pb file is greater than 50 mb so I am not able to push it , can I use git LFS now?? |
@SSahas You meant retrained_graph.pb? The current one in repo is already 83MB |
hey @ritwik12 , please tell if there are any things to be changed. |
Hey @ritwik12 , I'm interested in contributing to this issue. This is my first time contributing. I want to add a dwarf planet named 'Makemake' to the classification. |
Currently we only have 8 planets and it would be great to include Pluto and other dwarf planets too.
The text was updated successfully, but these errors were encountered: