Skip to content
This repository was archived by the owner on Jul 7, 2026. It is now read-only.

updated the requirements.txt#253

Open
deepthi1107 wants to merge 4 commits into
openMF:masterfrom
deepthi1107:master
Open

updated the requirements.txt#253
deepthi1107 wants to merge 4 commits into
openMF:masterfrom
deepthi1107:master

Conversation

@deepthi1107

Copy link
Copy Markdown

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

Param-29 commented Apr 9, 2023

Copy link
Copy Markdown
Collaborator

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

deepthi1107 commented Apr 9, 2023 via email

Copy link
Copy Markdown
Author

@deepthi1107

deepthi1107 commented Apr 10, 2023 via email

Copy link
Copy Markdown
Author

@deepthi1107

deepthi1107 commented Apr 10, 2023 via email

Copy link
Copy Markdown
Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants