Merged
Conversation
for more information, see https://pre-commit.ci
…dd/detect_multi_backend
109c5d6 to
0bfaba5
Compare
Member
Author
|
Ok this is PR is ready to merge! @zldrobit it doesn't make sense to include TensorFlow.js in here because this model needs a javascript environment to run right? |
…dd/detect_multi_backend
Contributor
|
@glenn-jocher I agree with you. It doesn't make sense, since TesnorFlow.js requires a browser or node.js environment to run and can only be called by javascript. |
1 task
This was referenced Nov 9, 2021
Merged
BjarneKuehl
pushed a commit
to fhkiel-mlaip/yolov5
that referenced
this pull request
Aug 26, 2022
* New `DetectMultiBackend()` class * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * pb to pt fix * Cleanup * explicit apply_classifier path * Cleanup2 * Cleanup3 * Cleanup4 * Cleanup5 * Cleanup6 * val.py MultiBackend inference * warmup fix * to device fix * pt fix * device fix * Val cleanup * COCO128 URL to assets * half fix * detect fix * detect fix 2 * remove half from DetectMultiBackend * training half handling * training half handling 2 * training half handling 3 * Cleanup * Fix CI error * Add torchscript _extra_files * Add TorchScript * Add CoreML * CoreML cleanup * New `DetectMultiBackend()` class * pb to pt fix * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Cleanup * explicit apply_classifier path * Cleanup2 * Cleanup3 * Cleanup4 * Cleanup5 * Cleanup6 * val.py MultiBackend inference * warmup fix * to device fix * pt fix * device fix * Val cleanup * COCO128 URL to assets * half fix * detect fix * detect fix 2 * remove half from DetectMultiBackend * training half handling * training half handling 2 * training half handling 3 * Cleanup * Fix CI error * Add torchscript _extra_files * Add TorchScript * Add CoreML * CoreML cleanup * revert default to pt * Add Usage examples * Cleanup val Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Consolidates all backends into a single YOLOv5
DetectMultiBackend()class. Supports following model formats:*.pt*.torchscript.pt*.mlmodelsaved_model*.pb*.tfliteweb_model*.onnx*.onnxUsage
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Refactoring and optimization of YOLOv5 inference and export.
📊 Key Changes
coco128.yamlfile.detect.pyby removing unused imports and refactoring model loading code.DetectMultiBackendclass tomodels/common.pyfor flexible backend support during inference.export.py.FP16) support and device selection improvements for inference code.DetectMultiBackendto include visualization features.detect.py,val.py, andexport.pyto utilize new backend class and streamline the codebase.load_classifierfunction fromutils/torch_utils.pyas it was deprecated.🎯 Purpose & Impact