Skip to content

Conversation

@glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Aug 29, 2022

The low package version is causing conflicts among other dependencies, commenting it causes no ill effects in CI so this should be fine.

Signed-off-by: Glenn Jocher [email protected]

๐Ÿ› ๏ธ PR Summary

Made with โค๏ธ by Ultralytics Actions

๐ŸŒŸ Summary

Improving compatibility by adjusting protobuf requirement.

๐Ÿ“Š Key Changes

  • The specific version restriction for protobuf has been removed from the requirements.txt file.

๐ŸŽฏ Purpose & Impact

  • ๐ŸŽˆ The purpose is to avoid potential conflicts with other packages that require different protobuf versions, enhancing the flexibility of the YOLOv5 installation.
  • ๐Ÿ”— This change could impact users by making it easier to integrate YOLOv5 with other software stacks that might need a newer version of protobuf.
  • ๐Ÿ‘ Users no longer need to downgrade protobuf manually for compatibility with YOLOv5, which streamlines the setup process.

The low package version is causing conflicts among other dependencies, commenting it causes no ill effects in CI so this should be fine.

Signed-off-by: Glenn Jocher <[email protected]>
@glenn-jocher glenn-jocher merged commit bd5fd78 into master Aug 29, 2022
@glenn-jocher glenn-jocher deleted the remove/protobuf branch August 29, 2022 15:13
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
The low package version is causing conflicts among other dependencies, commenting it causes no ill effects in CI so this should be fine.

Signed-off-by: Glenn Jocher <[email protected]>

Signed-off-by: Glenn Jocher <[email protected]>
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