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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Argparse support added. #188

Closed
wants to merge 2 commits into from
Closed

Argparse support added. #188

wants to merge 2 commits into from

Conversation

kadirnar
Copy link

@kadirnar kadirnar commented Aug 6, 2022

Feature Request

Type of change

  • 馃敩 New feature (non-breaking change which adds functionality)

Hi @aniketmaurya,

I added the argparser property for the image_classifier.py file.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

馃憢 @kadirnar! Thank you for opening your first pull request in this repo. We are so happy that you have decided to contribute and value your contribution. Please read these materials before proceeding: Contributing Guide and Code of Conduct.

@kadirnar
Copy link
Author

kadirnar commented Aug 7, 2022

Hi @aniketmaurya ,

Can you review the codes? The main code was already wrong. I just added argparse support.

@aniketmaurya
Copy link
Contributor

Hey @kadirnar, thanks for your contribution. I will fix the pytest and then merge your PR.
Also, it will take some time since I'm on sick leave. Thanks 馃槉

@kadirnar
Copy link
Author

kadirnar commented Aug 7, 2022

Hey @kadirnar, thanks for your contribution. I will fix the pytest and then merge your PR. Also, it will take some time since I'm on sick leave. Thanks 馃槉

Thanks. Get well soon

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2022

Stale pull request message

@github-actions github-actions bot closed this Oct 14, 2022
@kadirnar kadirnar deleted the argparse branch October 14, 2022 04:11
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.

None yet

2 participants