-
-
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 test cases #56
Comments
Hello! I would like to contribute to this project! But I am new to the open-source contribution, can you help me through the process? |
@shawngonsalves Hey, thanks for taking an interest. This project is mainly a Deep Learning Model. Part of it is to train and another is to evaluate. |
Hi! Is this issue open? I would like to work on it. |
@Shreya-Pathak Yep, this is open. Please go ahead!! |
I have increased the test_data for each of the classes and raised a PR. I obtained the images via Google search. Is there anything else to do? |
@Shreya-Pathak I have replied on the PR itself. Please take a look. |
hey, I m new to open source and want to contribute to this project can anyone help me, please. |
@Rahul-3000 Hey, Thanks for taking interest in this. Let me know what you need help with. |
@ritwik12 Hi, can you guide me how can I contribute i know python, cpp and html css |
@Rahul-3000 For this you need to have some Tensorflow and Deep Learning knowledge so that we can cover all the cases of test cases for the evaluation part or |
If you can go through the |
Hii @ritwik12, I am new to open source and looking for the issues labeled "good first issue". Can you help me with understanding this issue? exactly where do I have to make the changes? |
@AkshatGadhwal As of now, we have limited test cases. For this project to work better. We should cover up the edge cases where it can break and run those as part of the PR using Travis for every new patch of code that comes in. |
We need more test cases to make our Model better while receiving PRs. Test cases can be then included as part of Travis.
The text was updated successfully, but these errors were encountered: