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

Added training data for comets #32

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

kousikr26
Copy link

No description provided.

@ritwik12
Copy link
Owner

@kousikr26 Thanks for the training data, are you going to add the code change as well? Please refer to #26 if you need some reference.

@ritwik12 ritwik12 added the enhancement New feature or request label Oct 28, 2019
@kousikr26
Copy link
Author

@ritwik12 Added the code as well please review

Copy link
Owner

@ritwik12 ritwik12 left a comment

Choose a reason for hiding this comment

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

Please add test data and also update hub/examples/image_retraining/display_info.yml for the same @kousikr26

@kousikr26
Copy link
Author

@ritwik12 Added test data, the same problem for comets regarding the display_info.yml

@ritwik12
Copy link
Owner

ritwik12 commented Nov 5, 2019

@kousikr26 Please remove comet_2.jpg as test data should not be in training data and comet_3.jpgis having text on image which will be a NOISE for classification.
Also please check that test data and training data are not common and all images are real and not computarised. For image_5 .jpg also it will look more like a meteor as it is zoomed image.

@ritwik12 ritwik12 mentioned this pull request Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Needs work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants