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

updated the requirements.txt #253

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

Conversation

deepthi1107
Copy link

Fixes: issue #213 [updated requirements.txt]

Changes: I have changed the requirements.txt file according to #213

Screenshots for the change:
image

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run ./gradlew spotlessCheck to ensure the code formatting is maintained.

  • [x+] Run the unit tests with ./gradlew check to make sure you didn't break anything

@Param-29 Param-29 mentioned this pull request Apr 9, 2023
2 tasks
@Param-29
Copy link
Collaborator

Param-29 commented Apr 9, 2023

Great work!, could you merge your PR’s I see to many PRs and it misses the point of one PR for one task
I think you can merge PRs into the following 3

  1. Data Augmentation
    1. Add in the description of all the changes you are making Eg: Rotation, blur, etc etc
  2. Image classification
  3. Object detection
    1. Using YOLOv5s & CNN

This would help in getting the bigger picture of what changes you are making

The above is applicable for following PRs

#253-#301 (~50PRs in total)

@deepthi1107
Copy link
Author

deepthi1107 commented Apr 9, 2023 via email

@deepthi1107
Copy link
Author

deepthi1107 commented Apr 10, 2023 via email

@deepthi1107
Copy link
Author

deepthi1107 commented Apr 10, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants