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

Detection Updated #12

Merged
merged 1 commit into from
Nov 23, 2019
Merged

Detection Updated #12

merged 1 commit into from
Nov 23, 2019

Conversation

GOSHROW
Copy link
Contributor

@GOSHROW GOSHROW commented Nov 10, 2019

I have added configuration files for yolov3 as was told to. The weights could not be added for yolov3 as they exceeded file size. I have tried to add weights for yolov3tiny however it returned 403 access denied during push. I also tried up the NFPA dataset, however, the weights file is too large for the yolov3 architecture. furthermore,every epoch is computationally heavy for yolov3.

To avoid such failure in approach, I did in issue #5 , I added the approach through SSD-mobile net on coco and other datasets. these changes have been directly made on app.py than on detection. The automation for switching to SSD than yolov3 is left open for the indexer algorithm. However, SSD has provided more reliability in practical uses. The SSD related prototext and weight-config files may be seen in the repository: dataset.

The wieghts and cfg for yolov3 have been added to yolo-coco.

@ayulockin
Copy link
Member

Nice PR. I am reviewing this. Will let reach out to you for queries.

@ayulockin
Copy link
Member

Can you list down the major changes in the PR after your detailed introduction?

@GOSHROW
Copy link
Contributor Author

GOSHROW commented Nov 12, 2019

The major changes are :

  • SSD detection has been added as an alternative to the YOLOv3.
  • Prototxt and CaffeModels have been added for the SSD.
  • The switching and control-flow for detection models has not been added.
  • YOLO config files have been added to train upon.
  • YOLO coco pre-trained weights and cfg have been added.
  • pbtxt and config files have been added for SSD

@ayulockin ayulockin merged commit dea79c6 into thecodefoundation:master Nov 23, 2019
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