ppi-vision
ppi-vision copied to clipboard
Image classification yolov5 model
Fixes #298
Changes: Image classification yolov5 model
Screenshots for the change:
Please make sure these boxes are checked before submitting your pull request - thanks!
-
[x] Run
./gradlew spotlessCheckto ensure the code formatting is maintained. -
[x] Run the unit tests with
./gradlew checkto make sure you didn't break anything
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
- Data Augmentation
- Add in the description of all the changes you are making Eg: Rotation, blur, etc etc
- Image classification
- Object detection
- 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)
Hello Param,
I would do the changes and let you know.
Best regards, Deepthi M
On Sun, 9 Apr 2023 at 11:49 AM, Param Mirani @.***> wrote:
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
- Data Augmentation
- Add in the description of all the changes you are making Eg: Rotation, blur, etc etc
- Image classification
- Object detection
- 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 https://github.com/openMF/ppi-vision/pull/253-#301 https://github.com/openMF/ppi-vision/pull/301 (~50PRs in total)
— Reply to this email directly, view it on GitHub https://github.com/openMF/ppi-vision/pull/301#issuecomment-1501052027, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS3DRJOZSVRJEQ5GIBMYSYLXAJIGFANCNFSM6AAAAAAWL3MFAI . You are receiving this because you authored the thread.Message ID: @.***>
Following are the changes done as suggested by @Param Mirani https://mifos.slack.com/team/U04UG3A359N. Kindly check and do let me know for further changes and give feedback please. https://github.com/openMF/ppi-vision/pull/308 https://github.com/openMF/ppi-vision/pull/306 https://github.com/openMF/ppi-vision/pull/304
On Sun, 9 Apr 2023 at 16:02, Deepthi @.***> wrote:
Hello Param,
I would do the changes and let you know.
Best regards, Deepthi M
On Sun, 9 Apr 2023 at 11:49 AM, Param Mirani @.***> wrote:
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
- Data Augmentation
- Add in the description of all the changes you are making Eg: Rotation, blur, etc etc
- Image classification
- Object detection
- 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 https://github.com/openMF/ppi-vision/pull/253-#301 https://github.com/openMF/ppi-vision/pull/301 (~50PRs in total)
— Reply to this email directly, view it on GitHub https://github.com/openMF/ppi-vision/pull/301#issuecomment-1501052027, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS3DRJOZSVRJEQ5GIBMYSYLXAJIGFANCNFSM6AAAAAAWL3MFAI . You are receiving this because you authored the thread.Message ID: @.***>
commit name : PR link added object detection files : https://github.com/openMF/ppi-vision/pull/308 image classification using yolov5 and cnn layers: https://github.com/openMF/ppi-vision/pull/306 added data augmentation techniques : https://github.com/openMF/ppi-vision/pull/304 (edited)
On Mon, 10 Apr 2023 at 18:18, Deepthi @.***> wrote:
Following are the changes done as suggested by @Param Mirani https://mifos.slack.com/team/U04UG3A359N. Kindly check and do let me know for further changes and give feedback please. https://github.com/openMF/ppi-vision/pull/308 https://github.com/openMF/ppi-vision/pull/306 https://github.com/openMF/ppi-vision/pull/304
On Sun, 9 Apr 2023 at 16:02, Deepthi @.***> wrote:
Hello Param,
I would do the changes and let you know.
Best regards, Deepthi M
On Sun, 9 Apr 2023 at 11:49 AM, Param Mirani @.***> wrote:
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
- Data Augmentation
- Add in the description of all the changes you are making Eg: Rotation, blur, etc etc
- Image classification
- Object detection
- 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 https://github.com/openMF/ppi-vision/pull/253-#301 https://github.com/openMF/ppi-vision/pull/301 (~50PRs in total)
— Reply to this email directly, view it on GitHub https://github.com/openMF/ppi-vision/pull/301#issuecomment-1501052027, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS3DRJOZSVRJEQ5GIBMYSYLXAJIGFANCNFSM6AAAAAAWL3MFAI . You are receiving this because you authored the thread.Message ID: @.***>